Skip to content

Conversation

@costdev
Copy link
Contributor

@costdev costdev commented Jul 22, 2023

This PR contains commits separated by component for easier reviewing/possible commit.

Changes can be reviewed altogether or per PR commit.
If moved forward for merging to WordPress Core, this PR's commits can be merged individually, grouped into smaller chunks, or squashed into a single commit.

Trac ticket: https://core.trac.wordpress.org/ticket/58874

costdev added 28 commits July 22, 2023 06:26
@costdev costdev force-pushed the replace_isset_ternary_with_null_coalescing_operator branch from 11b83d3 to 51984d2 Compare July 22, 2023 05:27
@costdev costdev marked this pull request as ready for review July 22, 2023 05:40
Copy link
Member

@mukeshpanchal27 mukeshpanchal27 left a comment

Choose a reason for hiding this comment

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

Thanks @costdev for the PR. Could you please rebased?

@costdev
Copy link
Contributor Author

costdev commented Sep 21, 2023

@mukeshpanchal27 This PR is on hold until a Make post has been made to establish what PHP 7 syntaxes will be supported/allowed in Core.

@westonruter
Copy link
Member

Refresh: #10654

@github-actions
Copy link

The following accounts have interacted with this PR and/or linked issues. I will continue to update these lists as activity occurs. You can also manually ask me to refresh this list by adding the props-bot label.

Unlinked Accounts

The following contributors have not linked their GitHub and WordPress.org accounts: @me@colin-stewart.co.uk.

Contributors, please read how to link your accounts to ensure your work is properly credited in WordPress releases.

Core Committers: Use this line as a base for the props when committing in SVN:

Props mukesh27, costdev, westonruter.

To understand the WordPress project's expectations around crediting contributors, please review the Contributor Attribution page in the Core Handbook.

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