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

Docs: Update all local links in README files to work with npm #13030

Merged
merged 6 commits into from
Dec 20, 2018

Conversation

gziolo
Copy link
Member

@gziolo gziolo commented Dec 20, 2018

Description

This is a follow-up for #12835 where we discussed updating local link in README files to work with GitHub, npm and Gutenberg handbook. Thanks to changes introduced in https://meta.trac.wordpress.org/changeset/7989 and https://meta.trac.wordpress.org/changeset/7993 by @dd32, we are now able to use absolute GitHub paths to work universally in all 3 platforms 🎉

All urls in /packages folder were updated to avoid directory traversing.

@gziolo gziolo added this to the Documentation & Handbook milestone Dec 20, 2018
@gziolo gziolo added npm Packages Related to npm packages [Type] Developer Documentation Documentation for developers labels Dec 20, 2018
@gziolo gziolo self-assigned this Dec 20, 2018
packages/blocks/README.md Outdated Show resolved Hide resolved
@gziolo
Copy link
Member Author

gziolo commented Dec 20, 2018

I noticed some inconistent new lines added in blocks README file and I fixed them in e826988.

packages/nux/README.md Outdated Show resolved Hide resolved
@@ -1,6 +1,6 @@
# MenuItem

MenuItem is a component which renders a button intended to be used in combination with the [DropdownMenu component](../dropdown-menu).
MenuItem is a component which renders a button intended to be used in combination with the [DropdownMenu component](/packages/components/src/dropdown-menu/README.md).
Copy link
Member

Choose a reason for hiding this comment

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

Just noticed that the components links included /src/ which wasn't originally covered in #12835 - https://meta.trac.wordpress.org/changeset/7998 should cover that case too :)

@gziolo
Copy link
Member Author

gziolo commented Dec 20, 2018

@dd32, I update all links that wouldn't work. I went too far with updates :)

Copy link
Member

@ajitbohra ajitbohra left a comment

Choose a reason for hiding this comment

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

LGTM 👍

Seems like it would be good to add a note to https://github.com/WordPress/gutenberg/blob/master/CONTRIBUTING.md#contribute-to-the-documentation

On adding links to docs.

@gziolo gziolo merged commit e74f78b into master Dec 20, 2018
@gziolo gziolo deleted the update/docs-links-packages branch December 20, 2018 17:14
@gziolo
Copy link
Member Author

gziolo commented Dec 20, 2018

Once we confirm it works everywhere as we anticipate, we definitely should 😃

Are there any Markdown linters?

@ajitbohra
Copy link
Member

yes, we have many options we can zero down on one to cut down formatting reviews and bring in some consistency across docs.

https://github.com/markdownlint/markdownlint
https://github.com/remarkjs/remark-lint
https://github.com/DavidAnson/markdownlint

@chrisvanpatten
Copy link
Member

@gziolo See #12426 :)

@ajitbohra
Copy link
Member

@chrisvanpatten thanks for linking this was trying to recollect the same 👍

youknowriad pushed a commit that referenced this pull request Jan 9, 2019
* Docs: Update all local links in README files to work with npm

* Update packages/blocks/README.md

* Fix new lines in blocks README file

* Revert nux component link

* Revert link to the block library

* Update link to component from edit-post package
youknowriad pushed a commit that referenced this pull request Jan 9, 2019
* Docs: Update all local links in README files to work with npm

* Update packages/blocks/README.md

* Fix new lines in blocks README file

* Revert nux component link

* Revert link to the block library

* Update link to component from edit-post package
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
npm Packages Related to npm packages [Type] Developer Documentation Documentation for developers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants