Skip to content

Fix rwt link performance issues#2200

Merged
dac09 merged 6 commits intoredwoodjs:mainfrom
dac09:fix/rwt-link-again
Apr 2, 2021
Merged

Fix rwt link performance issues#2200
dac09 merged 6 commits intoredwoodjs:mainfrom
dac09:fix/rwt-link-again

Conversation

@dac09
Copy link
Copy Markdown
Contributor

@dac09 dac09 commented Apr 2, 2021

Try number 1500.

Tries to make yarn rwt link faster by only spinning up one watcher (rather than like 16).

Bonus:

  • Now supports --only flag to selectively link a certain package e.g.
yarn rwt link ~/Code/redwood --only router

Note that only is the folder name, not the package name. i.e. the part here -> @redwoodjs/<PACKAGE_FOLDER_NAME>

  • Much easier to link to any project directly from the framework (but only use this if you know what you're doing)
yarn build:link --dest ~/MyRedwoodProject/packages

This just does the build, you still need to install on the repo to actually link. This can be really useful in certain situations. Note that the path has packages because this is included by default in your workspaces

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Apr 2, 2021

📦 PR Packages

Click to Show Package Download Links

https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/create-redwood-app-0.28.4-586af8e.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/redwoodjs-api-0.28.4-586af8e.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/redwoodjs-api-server-0.28.4-586af8e.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/redwoodjs-auth-0.28.4-586af8e.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/redwoodjs-cli-0.28.4-586af8e.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/redwoodjs-core-0.28.4-586af8e.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/redwoodjs-dev-server-0.28.4-586af8e.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/redwoodjs-eslint-config-0.28.4-586af8e.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/redwoodjs-eslint-plugin-redwood-0.28.4-586af8e.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/redwoodjs-forms-0.28.4-586af8e.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/redwoodjs-internal-0.28.4-586af8e.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/redwoodjs-prerender-0.28.4-586af8e.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/redwoodjs-router-0.28.4-586af8e.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/redwoodjs-structure-0.28.4-586af8e.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/redwoodjs-testing-0.28.4-586af8e.tgz
https://rw-pr-redwoodjs-com.s3.amazonaws.com/2200/redwoodjs-web-0.28.4-586af8e.tgz

Install this PR by running yarn rw upgrade --pr 2200:0.28.4-586af8e

Copy link
Copy Markdown
Contributor

@thedavidprice thedavidprice left a comment

Choose a reason for hiding this comment

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

🙈

@dac09 dac09 merged commit 45667fe into redwoodjs:main Apr 2, 2021
@thedavidprice thedavidprice added this to the next release milestone Apr 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants