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

Improve monorepo task workflows #908

Merged
merged 1 commit into from
Feb 28, 2025
Merged

Conversation

necolas
Copy link
Contributor

@necolas necolas commented Feb 27, 2025

  • Faster task and CI builds by excluding 'app' workspace builds.
  • Faster code formatting by excluding large fixtures.
  • Automatic precommit checks for format and lint errors.
  • npm test runs the same tests (locally) as the CI workflow.
  • Minor reorganization of workflow jobs.

Fixes #878

@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Feb 27, 2025
@necolas necolas force-pushed the refactor-workflows-and-tasks branch from 4519b77 to 9890f5a Compare February 27, 2025 19:39
Copy link

vercel bot commented Feb 27, 2025

The latest updates on your projects. Learn more about Vercel for Git ↗︎

Name Status Preview Comments Updated (UTC)
stylex ❌ Failed (Inspect) Feb 28, 2025 6:48pm

@necolas
Copy link
Contributor Author

necolas commented Feb 27, 2025

The "size" benchmark only fails because the base commit doesn't have the new benchmarks package. If you inspect the job you can see that the "patch" step ran successfully. After this lands, future PRs will collect benchmark data correctly

Copy link
Member

@mellyeliu mellyeliu left a comment

Choose a reason for hiding this comment

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

Tested the precommit hooks locally and looks great, lint/prettier/flow process much simplified

* Faster task and CI builds by excluding 'app' workspace builds.
* Faster code formatting by excluding large fixtures.
* Automatic precommit checks for format and lint errors.
* 'npm test' runs all the same tests as the CI workflow.
* Minor reorganization of workflow jobs.
Copy link

workflow: benchmarks/size

Comparison of minified (terser) and compressed (brotli) size results, measured in bytes. Smaller is better.

[email protected] size:compare
node size/compare.js /tmp/tmp.rnGhisgSNW /tmp/tmp.0KCstxuwZa

Results Base Patch Ratio
stylex/lib/stylex.js
· compressed 985
· minified 3,154
stylex/lib/StyleXSheet.js
· compressed 1,266
· minified 3,776
benchmarks/size/.build/bundle.js
· compressed 567,170
· minified 10,232,457
benchmarks/size/.build/stylex.css
· compressed 100,609
· minified 755,721

@necolas necolas merged commit 790ca8b into main Feb 28, 2025
8 of 9 checks passed
@mellyeliu mellyeliu deleted the refactor-workflows-and-tasks branch February 28, 2025 20:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

GitHub Workflow for Typos has false positives
3 participants