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 upstream changes to test files from wordpress-develop #181

Merged
merged 12 commits into from
Feb 6, 2025

Conversation

desrosj
Copy link
Contributor

@desrosj desrosj commented Feb 6, 2025

This merges upstream changes to the test classes from https://github.com/WordPress/wordpress-develop/ so that they can be removed in WordPress/wordpress-develop#8269.

See https://core.trac.wordpress.org/ticket/42668.

Note: In order to ensure the tests run successfully, the changes from #180 are included in this branch. That PR should be merged first.

@desrosj desrosj self-assigned this Feb 6, 2025
@desrosj desrosj marked this pull request as ready for review February 6, 2025 16:04
Copy link

github-actions bot commented Feb 6, 2025

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 Committers: Use this line as a base for the props when committing in SVN:

Props desrosj, swissspidy.

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

@desrosj desrosj merged commit 90f7db9 into WordPress:master Feb 6, 2025
29 checks passed
@desrosj desrosj deleted the sync/tests-from-wp-develop branch February 6, 2025 17:39
@jrfnl
Copy link
Member

jrfnl commented Feb 6, 2025

Note: this is actually a PR where "props"/co-author tags should have been used as the copies of the upstream commits are now not credited to the people who were involved with them.

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