-
-
Notifications
You must be signed in to change notification settings - Fork 638
Phase 3: Prepare core package for workspace structure #1830
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
Changes from 1 commit
Commits
Show all changes
36 commits
Select commit
Hold shift + click to select a range
7745727
Phase 3: Prepare core package for workspace structure
AbanoubGhadban ce91f23
Fix CI failures: Update Knip config and test imports for workspace stβ¦
AbanoubGhadban 0182224
Fix package.json workspace configuration for publishing
AbanoubGhadban 6f08b49
Fix workspace configuration for publishing and CI
AbanoubGhadban 385df7f
Fix Knip configuration to ignore build output directories
AbanoubGhadban 4417277
Fix ESLint configuration for workspace package imports
AbanoubGhadban 3b34f34
Fix Jest configuration for workspace and testing library setup
AbanoubGhadban 5fd223e
Fix Jest setupFilesAfterEnv path resolution for CI
AbanoubGhadban ef7c6e6
Update Gemfile.lock and package.json for path corrections
AbanoubGhadban 4b94aba
Update script/convert for workspace structure
AbanoubGhadban 8cf9138
Remove setupFilesAfterEnv from Jest configuration and clean up converβ¦
AbanoubGhadban ee4487a
Update CircleCI configuration to enhance caching strategy for node_moβ¦
AbanoubGhadban 5eb784f
Update package.json to simplify link-source script by removing redundβ¦
AbanoubGhadban a9e1e01
Update link-source script in package.json to target the correct direcβ¦
AbanoubGhadban 3250a03
Update build output path to workspace package directory
AbanoubGhadban 660cab3
Complete workspace migration: move tests, scripts, and babel config
AbanoubGhadban eab8e2a
Refactor package.json scripts to utilize workspaces for improved commβ¦
AbanoubGhadban c1c6b45
Enhance package.json scripts for improved workspace management
AbanoubGhadban b34e161
Rename package to react-on-rails-workspace and streamline scripts in β¦
AbanoubGhadban 946698d
Update CircleCI configuration to streamline caching for node_modules β¦
AbanoubGhadban a784eb6
Update Knip configuration and package.json scripts for improved path β¦
AbanoubGhadban 87fe9cc
Revert unnecessary file extension changes
AbanoubGhadban 3e992bc
Add global Jest configuration and refactor Knip settings
AbanoubGhadban 20f2f0b
Update ESLint configuration to reflect new package structure
AbanoubGhadban dfc1275
Update configuration files to reflect package structure changes
AbanoubGhadban caf8d34
Update MONOREPO_MERGER_PLAN.md to reflect co-located test structure
AbanoubGhadban 325e4e8
Update documentation and references to reflect new package structure
AbanoubGhadban e0d1365
Update test command in convert script for improved execution
AbanoubGhadban fd728b3
Update Knip configuration and package.json scripts for improved execuβ¦
AbanoubGhadban 0a97c1e
Update test command in package.json for improved execution
AbanoubGhadban 86e0ab6
Update DIRECTORY_LICENSING.md to reflect new directory structure
AbanoubGhadban ceb15a6
Update DIRECTORY_LICENSING.md to reflect new directory structure and β¦
AbanoubGhadban 4331ea1
Update CONTRIBUTING.md to correct link formatting
AbanoubGhadban 40ccdb2
Update MONOREPO_MERGER_PLAN.md to specify new build output locations β¦
AbanoubGhadban c9c406f
Update MONOREPO_MERGER_PLAN.md and MONOREPO_MERGER_PLAN_REF.md to refβ¦
AbanoubGhadban 804a88d
Update MONOREPO_MERGER_PLAN.md and MONOREPO_MERGER_PLAN_REF.md to refβ¦
AbanoubGhadban File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,66 @@ | ||
| { | ||
| "name": "react-on-rails", | ||
| "version": "16.1.1", | ||
| "description": "react-on-rails JavaScript for react_on_rails Ruby gem", | ||
| "main": "../../node_package/lib/ReactOnRails.full.js", | ||
| "type": "module", | ||
| "exports": { | ||
| ".": { | ||
| "react-server": "../../node_package/lib/pro/ReactOnRailsRSC.js", | ||
| "node": "../../node_package/lib/ReactOnRails.node.js", | ||
| "default": "../../node_package/lib/ReactOnRails.full.js" | ||
| }, | ||
| "./client": "../../node_package/lib/ReactOnRails.client.js", | ||
| "./registerServerComponent/client": "../../node_package/lib/pro/registerServerComponent/client.js", | ||
| "./registerServerComponent/server": { | ||
| "react-server": "../../node_package/lib/pro/registerServerComponent/server.rsc.js", | ||
| "default": "../../node_package/lib/pro/registerServerComponent/server.js" | ||
| }, | ||
| "./wrapServerComponentRenderer/client": "../../node_package/lib/pro/wrapServerComponentRenderer/client.js", | ||
| "./wrapServerComponentRenderer/server": { | ||
| "react-server": "../../node_package/lib/pro/wrapServerComponentRenderer/server.rsc.js", | ||
| "default": "../../node_package/lib/pro/wrapServerComponentRenderer/server.js" | ||
| }, | ||
| "./RSCRoute": "../../node_package/lib/pro/RSCRoute.js", | ||
| "./RSCProvider": "../../node_package/lib/pro/RSCProvider.js", | ||
| "./ServerComponentFetchError": "../../node_package/lib/pro/ServerComponentFetchError.js" | ||
| }, | ||
coderabbitai[bot] marked this conversation as resolved.
Outdated
Show resolved
Hide resolved
|
||
| "peerDependencies": { | ||
| "react": ">= 16", | ||
| "react-dom": ">= 16", | ||
| "react-on-rails-rsc": "19.0.2" | ||
| }, | ||
| "peerDependenciesMeta": { | ||
| "react-on-rails-rsc": { | ||
| "optional": true | ||
| } | ||
| }, | ||
| "files": [ | ||
| "../../node_package/lib" | ||
| ], | ||
| "scripts": { | ||
| "build": "yarn run clean && yarn run tsc --declaration", | ||
| "build-watch": "yarn run clean && yarn run tsc --watch", | ||
| "clean": "rm -rf ../../node_package/lib", | ||
| "test": "jest ../../node_package/tests", | ||
| "type-check": "yarn run tsc --noEmit --noErrorTruncation" | ||
| }, | ||
| "repository": { | ||
| "type": "git", | ||
| "url": "git+https://github.com/shakacode/react_on_rails.git" | ||
| }, | ||
| "keywords": [ | ||
| "react", | ||
| "webpack", | ||
| "JavaScript", | ||
| "Ruby", | ||
| "on", | ||
| "Rails" | ||
| ], | ||
| "author": "justin.gordon@gmail.com", | ||
| "license": "MIT", | ||
| "bugs": { | ||
| "url": "https://github.com/shakacode/react_on_rails/issues" | ||
| }, | ||
| "homepage": "https://github.com/shakacode/react_on_rails#readme" | ||
| } | ||
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
File renamed without changes.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
|
|
@@ -66,7 +66,7 @@ | |
| }, | ||
| "scripts": { | ||
| "preinstall": "yarn run link-source && yalc add --link react-on-rails", | ||
| "link-source": "cd ../.. && yarn run build && yalc publish", | ||
| "link-source": "cd ../.. && yarn run build && cd packages/react-on-rails && yalc publish", | ||
|
Contributor
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Run root install before build to avoid broken preinstall
- "link-source": "cd ../.. && yarn run build && cd packages/react-on-rails && yalc publish",
+ "link-source": "cd ../.. && yarn && yarn run build && cd packages/react-on-rails && yalc publish",π€ Prompt for AI Agents |
||
| "lint": "cd ../.. && yarn run lint", | ||
| "format": "cd ../.. && yarn run nps format", | ||
| "test:js": "yarn run jest ./tests", | ||
|
|
||
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Build before yalc publish to ensure lib exists
The workflow publishes via yalc without building first. Insert a build step before publishing.
π Committable suggestion
π€ Prompt for AI Agents