Skip to content

Update sizebot to match name of CircleCI build job#15714

Merged
acdlite merged 1 commit intofacebook:masterfrom
acdlite:update-sizebot
May 22, 2019
Merged

Update sizebot to match name of CircleCI build job#15714
acdlite merged 1 commit intofacebook:masterfrom
acdlite:update-sizebot

Conversation

@acdlite
Copy link
Copy Markdown
Collaborator

@acdlite acdlite commented May 22, 2019

The sizebot scrapes the GitHub /statuses endpoint to get the latest CircleCI build number for master, in order to fetch the bundle size info for that build, which are stored as build artifacts. (There's gotta be a better way to do this, but that's what we have for now.) This updates the script to match the name of the updated CircleCI job that generates the bundle sizes.

The sizebot scrapes the GitHub `/statuses` endpoint to get the lastest
CircleCI build number for master, in order to fetch the bundle size
info for that build, which are stored as build artifacts. (There's gotta
be a better way to do this, but that's what we have for now.) This
updates the script to match the name of the updated CircleCI job that
generates the bundle sizes.
@acdlite acdlite force-pushed the update-sizebot branch 5 times, most recently from 1b113d1 to d0e37bd Compare May 22, 2019 22:06
@acdlite acdlite force-pushed the update-sizebot branch 3 times, most recently from 6f45c5b to 8a985d6 Compare May 22, 2019 23:05
@sizebot
Copy link
Copy Markdown

sizebot commented May 22, 2019

No significant bundle size changes to report.

Generated by 🚫 dangerJS

@acdlite acdlite merged commit d66c8f2 into facebook:master May 22, 2019
acdlite added a commit to acdlite/react that referenced this pull request May 29, 2019
Same as facebook#15714. I moved the artifacts step to a different job, so I
need to update the name in the sizebot script to match.
acdlite added a commit to acdlite/react that referenced this pull request May 29, 2019
Same as facebook#15714. I moved the artifacts step to a different job, so I
need to update the name in the sizebot script to match.
acdlite added a commit that referenced this pull request May 29, 2019
Same as #15714. I moved the artifacts step to a different job, so I
need to update the name in the sizebot script to match.
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