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

Inconsistent documentation on Parcel integration #1004

Closed
embiem opened this issue Jun 4, 2018 · 2 comments
Closed

Inconsistent documentation on Parcel integration #1004

embiem opened this issue Jun 4, 2018 · 2 comments

Comments

@embiem
Copy link

embiem commented Jun 4, 2018

Description

In the README under "parcel", it tells me to follow step 1 & 2 of getting started. There it advises to hot export the root component.

On the linked parcel example though, it tells me // DO NOT USE HOT! as a comment in the code.

Which one should I believe? The README or the Code Comment?

If anyone knows this, please let me know. I'd be glad to submit a PR to remove this inconsistency accordingly.

@theKashey
Copy link
Collaborator

Sorry, example was created before we made some changes into Parcel, and we forget to remove comment later.
Readme is correct.

timm-aym pushed a commit to timm-aym/react-hot-loader that referenced this issue Jun 5, 2018
@embiem
Copy link
Author

embiem commented Jun 5, 2018

Thanks for responding so quickly! <3

@embiem embiem closed this as completed Jun 5, 2018
pete-murphy added a commit to pete-murphy/purescript-parcel-example that referenced this issue Oct 10, 2018
Fix browser refresh issue (justinwoo#2) by exporting "hot" module from `src/Main.purs`. (I think this is related to gaearon/react-hot-loader#1004.)
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

No branches or pull requests

2 participants