Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update Abhängigkeiten auf php8.2 in README #225

Merged
merged 2 commits into from
Jan 14, 2024

Conversation

DO1JLR
Copy link
Contributor

@DO1JLR DO1JLR commented Oct 30, 2023

Beim testen des #223 ist mir aufgefallen, das zumindest für Debian bookworm die aktuelle php Version 8.2 ist. Und seit #216 gemerged wurde, scheint php8 der aktuelle stand zu sein

Auch ist mir aufgefallen, das nicht nur php-curl und php-xml für das lokale testen benötigt werden, sondern auch das php paket. Ich habe es also in der README ergänzt.

Eventuell findet ihr diese PR nützlich und wollt ihn mergen 😇

Beim testen des voc#223 ist mir aufgefallen, das zumindest für Debian bookworm die aktuelle php Version 8.2 ist.
Und seit voc#216 gemerged wurde, scheint php8 der aktuelle stand zu sein

Auch ist mir aufgefallen, das nicht nur ``php-curl`` und ``php-xml`` für das lokale testen benötigt werden, sondern auch das php paket. Ich habe es also in der README ergänzt.

Eventuell findet ihr diese PR nützlich und wollt ihn mergen?
Copy link
Member

@saerdnaer saerdnaer left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@saerdnaer saerdnaer merged commit 0fed56b into voc:master Jan 14, 2024
1 check failed
@DO1JLR DO1JLR deleted the requirements branch January 14, 2024 21:13
saerdnaer pushed a commit that referenced this pull request Jan 14, 2024
Beim testen des #223 ist mir aufgefallen, das zumindest für Debian bookworm die aktuelle php Version 8.2 ist.
Und seit #216 gemerged wurde, scheint php8 der aktuelle stand zu sein

Auch ist mir aufgefallen, das nicht nur ``php-curl`` und ``php-xml`` für das lokale testen benötigt werden, sondern auch das php paket. Ich habe es also in der README ergänzt.

Eventuell findet ihr diese PR nützlich und wollt ihn mergen?
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants