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

webpack config still references react-routing #770

Closed
nuclearspike opened this issue Jul 26, 2016 · 3 comments · Fixed by #829
Closed

webpack config still references react-routing #770

nuclearspike opened this issue Jul 26, 2016 · 3 comments · Fixed by #829
Assignees
Labels

Comments

@nuclearspike
Copy link

https://github.com/kriasoft/react-starter-kit/blob/master/tools/webpack.config.js#L52 (+ 2 more)

If everything works without having those paths be valid, do they really do anything?

@koistya
Copy link
Member

koistya commented Jul 26, 2016

It's considered to be a good practice to explicitly specify what
folders/files should be parsed by a Webpack loader.

On Wednesday, 27 July 2016, Paul [email protected] wrote:

https://github.com/kriasoft/react-starter-kit/blob/master/tools/webpack.config.js#L52
(+ 2 more)

If everything works without having those paths be valid, do they really do
anything?


You are receiving this because you are subscribed to this thread.
Reply to this email directly, view it on GitHub
#770, or mute the
thread
https://github.com/notifications/unsubscribe-auth/AAMCDlWXnTQ_3H083A1JePj6ZDmPps4qks5qZn40gaJpZM4JVngc
.

@nuclearspike
Copy link
Author

But react-routing is not in the package anymore and should now be 'universal-router' which is what I changed those references to in the config.

@koistya
Copy link
Member

koistya commented Jul 26, 2016

Ah, right, need to remove that line.

On Wednesday, 27 July 2016, Paul [email protected] wrote:

But react-routing is not in the package anymore and should now be
'universal-router' which is what I changed those references to in the
config.


You are receiving this because you commented.
Reply to this email directly, view it on GitHub
#770 (comment),
or mute the thread
https://github.com/notifications/unsubscribe-auth/AAMCDnEQI1ttwqRSKHuZkzA2A6a6CuNYks5qZn_1gaJpZM4JVngc
.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants