-
Notifications
You must be signed in to change notification settings - Fork 23
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
upkeep/102: bump min WP and PHP versions #111
Conversation
@Sidsector9 do you think we can also add PHP version in composer config: https://getcomposer.org/doc/06-config.md#platform |
292980d
to
44431c8
Compare
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.
The .nvmrc
file is empty, shouldn't we specify Node version?
941d352
to
47906a2
Compare
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.
@Sidsector9 After the latest change, both 3 e2e test workflows fail at the "Build" step, which looks unsafe for the future automated release workflows. Could you please check what was going wrong there?
Description of the Change
Closes #102
Changelog Entry
Credits
Props @vikrampm1 @Sidsector9
Checklist: