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

Fix pinned WordPress version #58

Merged
merged 7 commits into from
Aug 12, 2022
Merged

Conversation

alecgeatches
Copy link
Contributor

This is a PR to undo a change made to temporarily fix tests. At the time of writing, there is a file naming mismatch between WordPress' latest release and the latest version of the testing library pulled from tests/phpunit. This causes tests to fail, and can be fixed by specifying a release WordPress version.

Once WordPress 6.1 is released this should no longer be an issue. When that happens, ensure tests pass in this PR and merge.

See this discussion in this PR for details: #57 (comment).

@alecgeatches
Copy link
Contributor Author

alecgeatches commented Aug 11, 2022

@chriszarate This has been fixed in wp-env and tests are now fixed. I submitted a PR for this fix in WordPress/gutenberg#43020, but it ended up being fixed in a concurrent issue WordPress/gutenberg#42826 in a similar but more robust and better tested way. The original issue didn't yet exist when I first looked into solving this problem and unfortunately I was not aware of the overlap. Next time I'll run through issues a second time when I start actual development, especially when there's a fresh bug like that.

The WordPress version is now properly fixed to the phpunit version and we're good to unpin WordPress here.

@chriszarate
Copy link
Member

Next time I'll run through issues a second time when I start actual development, especially when there's a fresh bug like that.

Can't say I'd recommend that! 😂

Screen Shot 2022-08-11 at 11 43 39 AM

@alecgeatches alecgeatches merged commit 760758a into trunk Aug 12, 2022
@alecgeatches alecgeatches deleted the fix/pinned-wordpress-version branch August 12, 2022 03:34
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.

2 participants