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

Updates Documentation Links#103

Open
goodguyry wants to merge 3 commits into
mainfrom
doc-updates
Open

Updates Documentation Links#103
goodguyry wants to merge 3 commits into
mainfrom
doc-updates

Conversation

@goodguyry

Copy link
Copy Markdown
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.

@ostowe ostowe left a comment

Copy link
Copy Markdown
Contributor

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.

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

<!-- External links -->
[webpack_home]: https://webpack.js.org/

Copy link
Copy Markdown
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!

Comment thread dist/README.md
@@ -1,3 +0,0 @@
# Distribution directory

Copy link
Copy Markdown
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

Comment thread dist/index.js
@@ -1,3 +0,0 @@
#!/usr/bin/env node

require('./cli');

Copy link
Copy Markdown
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.

2 participants