Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

[In Progress] Modernize to PHP 8 WPCS 3 #69

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

Merged
gmazzap merged 21 commits into version/2 from development
Aug 29, 2023
Merged

[In Progress] Modernize to PHP 8 WPCS 3 #69

gmazzap merged 21 commits into version/2 from development
Aug 29, 2023

Conversation

@gmazzap
Copy link
Contributor

@gmazzap gmazzap commented Aug 25, 2023
edited
Loading

Please check if the PR fulfills these requirements

  • The commit message follows our guidelines
  • Tests for the changes have been added (for bug fixes/features)
  • Docs have been added/updated (for bug fixes/features)

What kind of change does this PR introduce? (Bug fix, feature, docs update, ...)

Feature

What is the current behavior? (You can also link to an open issue here)

See:

In a nutshell, make compatible with PHP 8+and its features and WPCS 3

What is the new behavior (if this is a feature change)?

Dependencies are updated, compatibility improved. Still work to do to cover all modern PHP 8 things.

Does this PR introduce a breaking change? (What changes might users need to make in their application due to this PR?)

Yes.

Other information:

The target version/2 branch will be the branch used to release the new major.

bueltge reacted with thumbs up emoji
antonioeatgoat and others added 21 commits February 24, 2023 12:15
Apparently `getDeclarationName` will never support T_FN.
The feature branch we were using has been merged.
@gmazzap gmazzap marked this pull request as ready for review August 25, 2023 23:28
@gmazzap gmazzap requested a review from Chrico August 25, 2023 23:32
@gmazzap gmazzap changed the title (削除) Development (削除ここまで) (追記) [In Progress] Modernize to PHP 8 WPCS 3 (追記ここまで) Aug 25, 2023
@gmazzap gmazzap requested a review from bueltge August 25, 2023 23:54
@gmazzap gmazzap merged commit 725b79b into version/2 Aug 29, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@bueltge bueltge bueltge approved these changes

@Chrico Chrico Chrico approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

AltStyle によって変換されたページ (->オリジナル) /