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

Reorganize monorepo scripts #953

Merged
merged 1 commit into from
Mar 19, 2025
Merged

Reorganize monorepo scripts #953

merged 1 commit into from
Mar 19, 2025

Conversation

necolas
Copy link
Contributor

@necolas necolas commented Mar 18, 2025

What changed / motivation ?

Minor reorganization of the repo's scripts and tools. Private package names aren't scoped to @stylexjs and the release tooling is moved up to the root package.

@necolas necolas requested a review from mellyeliu March 18, 2025 23:37
@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 Mar 18, 2025
Copy link

github-actions bot commented Mar 18, 2025

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.4PkNH9LUsz /tmp/tmp.2jjcDf9unL

Results Base Patch Ratio
stylex/lib/stylex.js
· compressed 985 985 1.00
· minified 3,154 3,154 1.00
stylex/lib/StyleXSheet.js
· compressed 1,266 1,266 1.00
· minified 3,776 3,776 1.00
benchmarks/size/.build/bundle.js
· compressed 537,611 537,611 1.00
· minified 7,435,904 7,435,904 1.00
benchmarks/size/.build/stylex.css
· compressed 100,609 100,609 1.00
· minified 755,721 755,721 1.00

@@ -11,7 +11,8 @@
"prettier": "prettier --write \"**/*.js\" \"**/*.ts\"",
"prettier:report": "prettier --check \"**/*.js\"",
"lint": "npm run lint:report -- --fix",
"lint:report": "eslint",
"lint:report": "eslint packages tools",
Copy link
Contributor Author

Choose a reason for hiding this comment

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

eslint wasn't linting

@@ -38,7 +39,7 @@
"prettier": "^3.5.0",
"prettier-plugin-hermes-parser": "^0.26.0",
"rimraf": "^5.0.10",
"typescript": "^5.3.3"
Copy link
Contributor Author

Choose a reason for hiding this comment

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

typescript doesn't seem to be used by any monorepo tooling

@necolas
Copy link
Contributor Author

necolas commented Mar 19, 2025

ping @mellyeliu

@@ -1,37 +1,3 @@
# @stylexjs/scripts
# scripts
Copy link
Member

Choose a reason for hiding this comment

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

what's the distinction we're making here between scripts and tools here? tools specifically as broader developer tooling unrelated to code logic?

Copy link
Contributor Author

@necolas necolas Mar 19, 2025

Choose a reason for hiding this comment

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

At the moment, everything left in scripts is installed as a bin in the dependent packages - so they're basically CLI tools used within some of the packages. The root tools are limited to managing monorepo-wide devx tasks like publishing to npm, precommit checks, etc. I'll add these notes to the READMEs

@necolas necolas merged commit 5c527bb into main Mar 19, 2025
8 checks passed
@necolas necolas deleted the repo-scripts branch March 19, 2025 19:19
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.

3 participants