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

Composer: up the minimum PHPCS version to 3.7.1 #742

Merged
merged 1 commit into from
Jan 31, 2023

Conversation

GaryJones
Copy link
Contributor

Add support for PHP 8.1 features, including fixing a retokenization of reserved keywords bug in PHPCS 3.7.0.

With a new minimum version of PHPCS, this allows consolidation and simplification of the testing matrix in the test.yml workflow.

I'd like to get this into 2.3.4, but this may be semantically better for 3.0.0?

Add support for PHP 8.1 features, including fixing a retokenization of reserved keywords bug in PHPCS 3.7.0.

With a new minimum version of PHPCS, this allows consolidation and simplification of the testing matrix in the `test.yml` workflow.
@GaryJones GaryJones self-assigned this Jan 24, 2023
@GaryJones GaryJones requested a review from a team as a code owner January 24, 2023 16:35
@GaryJones
Copy link
Contributor Author

@rebeccahum Any preference on whether this could go into a 2.3.4, or hold on to a 3.0?

Copy link
Collaborator

@jrfnl jrfnl left a comment

Choose a reason for hiding this comment

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

Just left some questions, no biggie.

.github/workflows/test.yml Show resolved Hide resolved
.github/workflows/test.yml Show resolved Hide resolved
.github/workflows/test.yml Show resolved Hide resolved
@jrfnl
Copy link
Collaborator

jrfnl commented Jan 27, 2023

As for 2.3.4 or 3.0.0 - semantically, it's allowed in either, so I have no opinion on where it should go (other than my earlier remark that it may be useful to do a last 2.x release before develop becomes 3.x).

@GaryJones GaryJones merged commit df9caf9 into develop Jan 31, 2023
@GaryJones GaryJones deleted the feature/update-phpcs-min-version branch January 31, 2023 22:00
@jrfnl jrfnl added this to the 2.3.4 milestone Mar 5, 2023
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.

3 participants