Skip to content

[Danger] Minor fixes #12606

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.

Sign up for GitHub

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

Merged
gaearon merged 4 commits into from
Apr 11, 2018
Merged

[Danger] Minor fixes #12606

Changes from 1 commit
Commits
Show all changes
4 commits
Select commit Hold shift + click to select a range
379aea8
Don't download bundle stats from master on CI
gaearon Apr 11, 2018
0b79132
Use absolute threshold for whether to show a change
gaearon Apr 11, 2018
7430ce4
Download master stats, but only for other master builds
gaearon Apr 11, 2018
a21f43a
Rewrite sizes
gaearon Apr 11, 2018
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Download master stats, but only for other master builds
  • Loading branch information
@gaearon
gaearon committed Apr 11, 2018
commit 7430ce4b0c43f7ccacf84fafbe42791fe70b62a3
8 changes: 8 additions & 0 deletions scripts/circleci/build.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,14 @@

set -e

# On master, download the bundle sizes from last master build so that
# the size printed in the CI logs for master commits is accurate.
# We don't do it for pull requests because those are compared against
# the merge base by Dangerfile instead. See https://github.com/facebook/react/pull/12606.
if [ -z "$CI_PULL_REQUEST" ]; then
curl -o scripts/rollup/results.json http://react.zpao.com/builds/master/latest/results.json
fi

yarn build --extract-errors
# Note: since we run the full build including extracting error codes,
# it is important that we *don't* reset the change to `scripts/error-codes/codes.json`.
Expand Down