-
Notifications
You must be signed in to change notification settings - Fork 13
allow for custom sides in CLI dev script #177
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
base: main
Are you sure you want to change the base?
Conversation
✅ Deploy Preview for cedarjs canceled.
|
| expect(generateCommand.command).toEqual('yarn rw-gen-watch') | ||
| }) | ||
|
|
||
| it('Should run custom side', async () => { |
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.
Thanks for adding a test 🙏
|
Not saying we should do it like this – just a question: Could we look at the root "workspaces": {
"packages": [
"api",
"web"
]
},So to add more sides you add another package to the list there. |
Most of the times the workspaces are defined like |
|
This made me realize something: Currently when you run |
If it's added to the workspaces yes. |
Co-authored-by: Tobbe Lundberg <tobbe@tlundberg.com>
This allows to run additional scripts when executing
yarn rw dev.This is enabled by updating the
redwood.toml, e.g.Now when you run
yarn rw devit will also execute the android script present in the app workspace.It also allows to run
yarn rw dev appwich will only run the app side.I'm not completely set on this configuration, let me know your opinion.
One small issue with this is that if you have interactive CLI scripts they will not work, cannot really be fixed with the solution in place right now.