Skip to content

Conversation

poteto
Copy link
Member

@poteto poteto commented Mar 17, 2023

CircleCI now enforces passing a token when fetching artifacts. I'm also deleting the old request-promise-json dependency because AFAIK we were only using it to fetch json from circleci about the list of available artifacts – which we can just do using node-fetch. Plus, the underlying request package it uses has been deprecated since 2019.

@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Mar 17, 2023
@poteto poteto force-pushed the lt/fix-sizebot branch 6 times, most recently from 7ef111d to a509562 Compare March 17, 2023 19:47
@tyao1
Copy link
Contributor

tyao1 commented Mar 17, 2023

Thanks for fixing!! download_base_build_for_sizebot is failing, is it supposed to work on this PR?

CircleCI now enforces passing a token when fetching artifacts.
@poteto poteto marked this pull request as draft March 17, 2023 19:53
This package has been deprecated since 2019, we can use node-fetch
instead.
@poteto poteto marked this pull request as ready for review March 17, 2023 20:00
@poteto poteto requested review from tyao1 and kassens March 17, 2023 20:00
@react-sizebot
Copy link

Comparing: 6854a3c...0d286b1

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 = 157.36 kB 157.36 kB = 50.16 kB 50.16 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 158.94 kB 158.94 kB = 50.67 kB 50.67 kB
facebook-www/ReactDOM-prod.classic.js = 541.83 kB 541.83 kB = 96.43 kB 96.43 kB
facebook-www/ReactDOM-prod.modern.js = 525.55 kB 525.55 kB = 93.96 kB 93.96 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 0d286b1

@poteto poteto merged commit 6310087 into main Mar 17, 2023
@poteto poteto deleted the lt/fix-sizebot branch March 17, 2023 20:04
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.

4 participants