Skip to content

Conversation

GioLogist
Copy link
Contributor

@GioLogist GioLogist commented Nov 29, 2022

Improved DevX with Yarn Workspaces

  • Setup yarn workspaces, so that example package can import relay-nextjs directly
  • Update dependencies in relay-nextjs (dev only)
  • Update typescript with live reloading, for faster devx
  • Update docs with "contributing" instructions for developing locally to contribute

Contributing

If you'd like to contribute to this package, developing locally is now as simple as:

  1. Run yarn from the root of this package
  2. Start the live reloader for the package cd packages/relay-nextjs && yarn build-w
  3. In a separate terminal strt the example app: cd example && yarn dev

Now, when you make any changes in packages/relay-nextjs, they will instantly be reflected in the example app.

README.md Outdated
Comment on lines 291 to 301
## Contributing

If you'd like to contribute to this package, developing locally is simple.

1. Run `yarn` from the root of this package
2. Start the live reloader for the package `cd packages/relay-nextjs && yarn build-w`
3. In a separate terminal strt the example app: `cd example && yarn dev`

Now, when you make any changes in `packages/relay-nextjs`, they will instantly be reflected in the example app.
Copy link
Contributor

Choose a reason for hiding this comment

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

This is an awesome change! Thank you very much for the improvements here, it's not something I've spent a ton of time on as this repo is synced in and out of the Revere monorepo whenever changes are made. Can we add a note of which yarn version is required?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This is an awesome change! Thank you very much for the improvements here, it's not something I've spent a ton of time on as this repo is synced in and out of the Revere monorepo whenever changes are made.

No worries, happy to help!

Can we add a note of which yarn version is required?

Good call! Done.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@rrdelaney Anything else needed here btw?

Copy link
Contributor

Choose a reason for hiding this comment

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

Not at all! This PR is great, I just need to check it out locally and give it a whirl. Unfortunately I've been quite sick the past week so getting some time for that has been difficult. I will look soon though, no action need on your end, and thank you again for this wonderful effort.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Right on, hope you feel better soon 🤙🏻

@rrdelaney rrdelaney merged commit 986888e into RevereCRE:main Dec 16, 2022
rrdelaney added a commit that referenced this pull request May 16, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants