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

Codemod it.experimental to gate pragma #18582

Merged
merged 2 commits into from Apr 13, 2020

Conversation

acdlite
Copy link
Collaborator

@acdlite acdlite commented Apr 12, 2020

Codemods all uses of it.experimental to the gate pragma added in #18581.

Quick review link

@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Apr 12, 2020
@codesandbox-ci
Copy link

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 284730b:

Sandbox Source
practical-moser-blv35 Configuration

@codesandbox-ci
Copy link

codesandbox-ci bot commented Apr 12, 2020

This pull request is automatically built and testable in CodeSandbox.

To see build info of the built libraries, click here or the icon next to each commit SHA.

Latest deployment of this branch, based on commit 3183d7d:

Sandbox Source
magical-tu-bm4b5 Configuration

@sizebot
Copy link

sizebot commented Apr 12, 2020

No significant bundle size changes to report.

Size changes (stable)

Generated by 🚫 dangerJS against 3183d7d

@sizebot
Copy link

sizebot commented Apr 12, 2020

No significant bundle size changes to report.

Size changes (experimental)

Generated by 🚫 dangerJS against 3183d7d

@acdlite
Copy link
Collaborator Author

acdlite commented Apr 12, 2020

It's annoying how many whitespace changes there are, because the line length went down. But this illustrates the advantage of the pragma form: adding or removing a gate doesn't affect the whitespace, because the pragma goes on a separate line.

@acdlite acdlite force-pushed the remove-it-experimental branch 3 times, most recently from 9e69f02 to 8ecf4a2 Compare April 12, 2020 23:22
Find-and-replace followed by Prettier
Removes the API from our test setup script
@acdlite acdlite merged commit 65237a2 into facebook:master Apr 13, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed React Core Team Opened by a member of the React Core Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants