-
Notifications
You must be signed in to change notification settings - Fork 16
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
PHP 8 support and migration to stable CI system #24
Conversation
Also, we Drop support for old PHP version PHP with version below 7.3 isn't supported by PHP anymore. I think it's good idea to drop it too https://www.php.net/supported-versions.php
According to https://man7.org/linux/man-pages/man7/glob.7.html, now inverse search should using apostrhope instead of caret
…s at this point in time
…` is compatible with lowest supported version
Overall looks good: will need to adjust repo configs to merge and later add release automation, for which I still need admin access (pending) |
…version constraint Co-authored-by: Martins Sipenko <martins.sipenko@gmail.com>
A bunch of canned CI badges could be removed from |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great work!
…more stable environment
This is #21, but with added CI and maintenance tools that I need to take over maintenance here
Fixes #21
Fixes #17