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

Stop tracking bundle sizes #15404

Merged
merged 2 commits into from
Apr 12, 2019
Merged

Conversation

acdlite
Copy link
Collaborator

@acdlite acdlite commented Apr 12, 2019

Deletes scripts/rollup/results.json from the repo and adds the path .gitignore. After running the build script, CI will move the file to build/ before storing as an artifact. This is an extra precaution to ensure that sizebot compares against the correct sizes.

@sizebot
Copy link

sizebot commented Apr 12, 2019

Warnings
⚠️

Could not find build artifacts for base commit: ed67984

Generated by 🚫 dangerJS

@acdlite
Copy link
Collaborator Author

acdlite commented Apr 12, 2019

Looks like it worked:

Screen Shot 2019-04-12 at 1 32 40 PM

@acdlite acdlite merged commit 38bd570 into facebook:master Apr 12, 2019
threepointone pushed a commit to threepointone/react that referenced this pull request Apr 15, 2019
* [sizebot] Fail gracefully if CI returns invalid response

Moves the `response.json()` call into the catch block.

* Stop tracking bundle sizes
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.

3 participants