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

Merge trunk into auto-sizes #970

Merged
merged 109 commits into from
Feb 6, 2024
Merged

Merge trunk into auto-sizes #970

merged 109 commits into from
Feb 6, 2024

Conversation

mukeshpanchal27
Copy link
Member

Checklist

  • PR has either [Focus] or Infrastructure label.
  • PR has a [Type] label.
  • PR has a milestone or the no milestone label.

mukeshpanchal27 and others added 30 commits September 23, 2022 14:38
`slevomat/coding-standard` requires PHP 7.2+
…ction

Fix construction of translation strings in admin/plugins.php
westonruter and others added 11 commits January 31, 2024 15:19
Run build-cs composer install/update via post-install/update-cmd scripts
Props Bot is a new GitHub Action that will compile a list of contributors for a given pull request. The bot will leave a comment with a list of contributors formatted for use in both Trac SVN and GitHub. See: https://make.wordpress.org/core/2024/02/01/new-commit-message-requirements-in-git-hello-props-bot/
Introduce Props Bot workflow.

Co-authored-by: aaronjorbin <[email protected]>
Co-authored-by: swissspidy <[email protected]>
Co-authored-by: westonruter <[email protected]>
Replace leafs with leaves to appease Spell Check with Typos workflow
This improves the logic controlling when Props Bot runs to:
- not run for draft PRs.
- not run when comments are deleted (this should hardly ever happen).
- not run on closed PRs.
Improve conditions Props Bot runs under.
Update test to use non-AVIF format since now supported in 6.5-alpha

Co-authored-by: westonruter <[email protected]>
Co-authored-by: adamsilverstein <[email protected]>
Co-authored-by: joemcgill <[email protected]>
@mukeshpanchal27 mukeshpanchal27 added [Type] Enhancement A suggestion for improvement of an existing feature Infrastructure Issues for the overall performance plugin infrastructure no milestone PRs that do not have a defined milestone for release labels Feb 6, 2024
@mukeshpanchal27
Copy link
Member Author

Uni tests failing because of leafs.jpg is now leaves.jpg

Copy link

github-actions bot commented Feb 6, 2024

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.

Core SVN

If you're a Core Committer, use this list when committing to wordpress-develop in SVN:

Props: mukesh27, spacedmonkey, swissspidy, westonruter, flixos90, aristath.

GitHub Merge commits

If you're merging code through a pull request on GitHub, copy and paste the following into the bottom of the merge commit message.

Co-authored-by: mukeshpanchal27 <[email protected]>
Co-authored-by: spacedmonkey <[email protected]>
Co-authored-by: swissspidy <[email protected]>
Co-authored-by: westonruter <[email protected]>
Co-authored-by: felixarntz <[email protected]>
Co-authored-by: aristath <[email protected]>

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

@mukeshpanchal27
Copy link
Member Author

The unit tests have been fixed in the follow-up PR #971. Once these changes are committed, the follow-up PR will be rebased and then committed.

@swissspidy swissspidy merged commit 2fab10a into feature/auto-sizes Feb 6, 2024
101 of 111 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Infrastructure Issues for the overall performance plugin infrastructure no milestone PRs that do not have a defined milestone for release [Type] Enhancement A suggestion for improvement of an existing feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants