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

Delete unused Circle CI scripts. #22710

Merged
merged 1 commit into from
Nov 6, 2021

Conversation

elas7
Copy link
Contributor

@elas7 elas7 commented Nov 6, 2021

This PR deletes several Circle CI script files from /scripts/circleci and /scripts/release that are not used anymore.

Deleted files:

@sizebot
Copy link

sizebot commented Nov 6, 2021

Comparing: ee06906...e2dcb5c

Critical size changes

Includes critical production bundles, as well as any change greater than 2%:

Name +/- Base Current +/- gzip Base gzip Current gzip
oss-stable/react-dom/cjs/react-dom.production.min.js = 129.55 kB 129.55 kB = 41.43 kB 41.43 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 134.52 kB 134.52 kB = 42.89 kB 42.89 kB
facebook-www/ReactDOM-prod.classic.js = 423.36 kB 423.36 kB = 78.05 kB 78.05 kB
facebook-www/ReactDOM-prod.modern.js = 411.92 kB 411.92 kB = 76.30 kB 76.30 kB
facebook-www/ReactDOMForked-prod.classic.js = 423.36 kB 423.36 kB = 78.06 kB 78.06 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against e2dcb5c

@gaearon gaearon merged commit 327d5c4 into facebook:main Nov 6, 2021
@gaearon
Copy link
Collaborator

gaearon commented Nov 6, 2021

thanks

@elas7 elas7 deleted the delete-unused-circle-ci-scripts branch November 6, 2021 14:23
zhengjitf pushed a commit to zhengjitf/react that referenced this pull request Apr 15, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants