Skip to content

[ci] fix notify/label actions for forks #32725

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

Merged
merged 1 commit into from
Mar 24, 2025

Conversation

rickhanlonii
Copy link
Member

@rickhanlonii rickhanlonii commented Mar 24, 2025

Need this to run against target for forks to get the notification.

This job does not checkout the code in the PR, so it's safe to run from the target.

Also fixes failing checks on PRs:

Screenshot 2025-03-24 at 3 28 30 PM

@react-sizebot
Copy link

react-sizebot commented Mar 24, 2025

Comparing: 254dc4d...073bea0

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.js = 6.68 kB 6.68 kB = 1.83 kB 1.83 kB
oss-stable/react-dom/cjs/react-dom-client.production.js = 515.14 kB 515.14 kB = 91.74 kB 91.74 kB
oss-experimental/react-dom/cjs/react-dom.production.js = 6.69 kB 6.69 kB +0.05% 1.83 kB 1.83 kB
oss-experimental/react-dom/cjs/react-dom-client.production.js = 614.35 kB 614.35 kB = 108.76 kB 108.76 kB
facebook-www/ReactDOM-prod.classic.js = 650.34 kB 650.34 kB = 114.79 kB 114.79 kB
facebook-www/ReactDOM-prod.modern.js = 640.62 kB 640.62 kB = 113.23 kB 113.22 kB

Significant size changes

Includes any change greater than 0.2%:

(No significant changes)

Generated by 🚫 dangerJS against 073bea0

@rickhanlonii rickhanlonii force-pushed the rh/fix-notift-target branch from d9e3553 to 073bea0 Compare March 24, 2025 19:29
@rickhanlonii rickhanlonii changed the title [ci] fix notify for forks [ci] fix notify/label actions for forks Mar 24, 2025
@rickhanlonii rickhanlonii merged commit 2d40460 into facebook:main Mar 24, 2025
239 checks passed
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.

4 participants