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

[DiffTrain] Add artifacts for React Native to compiled #26204

Merged
merged 17 commits into from
Mar 20, 2023

Conversation

sammy-SC
Copy link
Contributor

No description provided.

@sammy-SC sammy-SC requested a review from poteto February 20, 2023 15:27
@facebook-github-bot facebook-github-bot added CLA Signed React Core Team Opened by a member of the React Core Team labels Feb 20, 2023
@sammy-SC sammy-SC marked this pull request as draft February 20, 2023 15:27
@react-sizebot
Copy link

react-sizebot commented Feb 20, 2023

Comparing: 2cc54b5...fa22b2c

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 = 154.55 kB 154.55 kB = 48.83 kB 48.83 kB
oss-experimental/react-dom/cjs/react-dom.production.min.js = 156.55 kB 156.55 kB = 49.46 kB 49.46 kB
facebook-www/ReactDOM-prod.classic.js = 531.43 kB 531.43 kB = 94.72 kB 94.72 kB
facebook-www/ReactDOM-prod.modern.js = 515.39 kB 515.39 kB = 92.26 kB 92.26 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against fa22b2c

ls -R ./compiled
- name: Move relevant files for React in fbsource into compiled
run: |
BASE_FOLDER='compiled/facebook-fbsource/xplat/js'
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do you mind making this a top level, sibling dir outside of ./compiled, eg ./compiled-rn? Reason being that the commit job just pushes the entire compiled dir to www and it'd be easier to separate them

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

sure, I can change it. But it is already separated during commit step where only facebook-www folder is synced to react repo. The step for commit support file_pattern.

@sammy-SC sammy-SC requested a review from poteto February 20, 2023 22:40
.github/workflows/commit_artifacts.yml Outdated Show resolved Hide resolved
.github/workflows/commit_artifacts.yml Outdated Show resolved Hide resolved
.github/workflows/commit_artifacts.yml Outdated Show resolved Hide resolved
@sammy-SC sammy-SC marked this pull request as ready for review March 17, 2023 18:30
@sammy-SC sammy-SC merged commit c57b90f into facebook:main Mar 20, 2023
@sammy-SC sammy-SC deleted the difftrain-add-fbsource branch March 20, 2023 11:24
bigfootjon pushed a commit that referenced this pull request Apr 18, 2024
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