Skip to content
This repository was archived by the owner on Jun 3, 2021. It is now read-only.

Conversation

@goodguyry
Copy link
Member

I noticed some links are broken, and others are a bit outdated.

I've updated the broken and outdated links throughout and reformatted them in an effort to make them more easily-updated going forward.

Copy link
Contributor

@ostowe ostowe left a comment

Choose a reason for hiding this comment

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

💯 address comments before merging, otherwise looks good.

* [Dist][dist_docs] - General information on the distribution (build) directory.

<!-- External links -->
[webpack_home]: https://webpack.js.org/
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 awesome, had no idea you could do this!

@@ -1,3 +0,0 @@
# Distribution directory
Copy link
Contributor

Choose a reason for hiding this comment

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

Add this info to CONTRIBUTING.md instead if it isn't already in there

@@ -1,3 +0,0 @@
#!/usr/bin/env node

require('./cli');
Copy link
Contributor

Choose a reason for hiding this comment

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

Pretty sure we need to keep this actually, forgot about that. CLI won't work without it.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants