Skip to content

Update hot-module-replacement.md #1454

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

Closed
wants to merge 1 commit into from
Closed

Conversation

evolve2k
Copy link

@evolve2k evolve2k commented Jul 26, 2017

Progressing through the tutorial so far, each tute lead on nicely to the next.
This one however breaks the flow and provided no clues as to which lines to add like previous files do.

Basically the convention of the green lines beginning with + signs is not present in this file, breaking the flow of the tutorial at this point.

I've marked up a couple of key ones that would be useful to highlight.
This is helpful as is it is but could probably do with more work also.

Progressing through the tutorial so far, each tute lead on nicely to the next.
This one however breaks the flow and provided no clues as to which lines to add like previous files do.
I've marked up a couple of key ones that would be useful to highlight.
This is helpful as is it is but could probably do with more work also.
@jsf-clabot
Copy link

jsf-clabot commented Jul 26, 2017

CLA assistant check
All committers have signed the CLA.

@TheDutchCoder
Copy link
Collaborator

Thanks for your contribution!

We're actually in the process of rewriting this guide, as you can see in this PR

I would suggest giving your input on that one if you like.

I'll close this PR in the mean time ;)

@evolve2k
Copy link
Author

Ah excellent, I was hoping that was the case. Thanks for your ongoing work on this great project!

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.

3 participants