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

Revert "theme.json: adds a setting property that enables some other ones" #36477

Merged
merged 1 commit into from Nov 15, 2021

Conversation

noisysocks
Copy link
Member

@noisysocks noisysocks commented Nov 15, 2021

Sorry @oandregal, merging this caused legitimate PHP unit test failures on trunk. I'm not sure why they were passing in the PR. Please revert this revert, fix up the tests, and I'll backport that into the 5.9 branch in time for Beta 2.

Reverts #36246

@noisysocks noisysocks added the Backport to WP Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Nov 15, 2021
@noisysocks noisysocks merged commit 94f0cb1 into trunk Nov 15, 2021
@noisysocks noisysocks deleted the revert-36246-add/theme-json-setting-to-opt-in branch November 15, 2021 04:39
@github-actions github-actions bot added this to the Gutenberg 12.0 milestone Nov 15, 2021
noisysocks added a commit that referenced this pull request Nov 15, 2021
@noisysocks noisysocks removed the Backport to WP Beta/RC Pull request that needs to be backported to the WordPress major release that's currently in beta label Nov 15, 2021
oandregal added a commit that referenced this pull request Nov 19, 2021
@oandregal
Copy link
Member

Second try at #36646

@oandregal
Copy link
Member

#36646 has been merged.

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.

None yet

2 participants