Skip to content

docs(concepts): Direct new-comers to read about JS modules #1773

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

Merged
merged 3 commits into from
Jan 16, 2018

Conversation

tamirSwag
Copy link
Contributor

Added a direction to learn about JS modules when reading the concepts page, meant mostly for new-comers. Because the modules doc page does not discuss the JS module systems, I also added a further reading link on this subject.

This is after a colleague of mine had an issue understanding what and why about webpack when reading the docs. What he was missing is an understanding of JS modules. You can't understand a module bundler without knowing modules.

This is because new-comers may have an easier time to understand webpack and why it is needed when they read about JS modules.
The article describes and discusses module systems in JS.
Copy link
Collaborator

@skipjack skipjack left a comment

Choose a reason for hiding this comment

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

Great additions, however see my comment re one of the links you added. Once you address it I can merge.

@@ -12,6 +12,8 @@ contributors:

At its core, *webpack* is a _static module bundler_ for modern JavaScript applications. When webpack processes your application, it recursively builds a _dependency graph_ that includes every module your application needs, then packages all of those modules into one or more _bundles_.

T> Learn more about JavaScript modules and webpack modules [here](/modules).
Copy link
Collaborator

Choose a reason for hiding this comment

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

Shouldn't the link be to /concepts/modules -- e.g. here.

@skipjack skipjack merged commit 5e86e4e into webpack:master Jan 16, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants