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: url fix #256

Merged
merged 3 commits into from
May 4, 2019
Merged

docs: url fix #256

merged 3 commits into from
May 4, 2019

Conversation

Berkmann18
Copy link
Member

What:
Code comment the localhost URL and fix some minor typos.

Why:
Because the URL is treated as a live one ending up in yet another entry in the broken URLs (re #187) and some typos which got flagged by something.

How:
...

Checklist:

  • Documentation
  • Ready to be merged
  • Added myself to contributors table.
    Bot Usage

@Berkmann18 Berkmann18 added the documentation Documentation related issue/PR label Apr 15, 2019
@Berkmann18 Berkmann18 self-assigned this Apr 15, 2019
@@ -64,7 +64,7 @@ All Contributors uses Docusaurus to maintain our documentation website. For more
To get started locally:
- `yarn install`
- `yarn start`
- go to http://localhost:3000 (Refresh the page for your changes to be reflected)
- go to `http://localhost:3000` (Refresh the page for your changes to be reflected)
Copy link
Contributor

Choose a reason for hiding this comment

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

prefer to change to markdown link

Copy link
Member Author

Choose a reason for hiding this comment

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

@sinchang The problem with that is the fact that it will be treated as a live URL which would inevitably return noise on brokenlinkcheck which is what I tried to avoid doing this.

Copy link
Member Author

Choose a reason for hiding this comment

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

@sinchang Do you have a better idea regarding that?

@Berkmann18 Berkmann18 added this to In progress in All Contributors Kanban via automation Apr 16, 2019
All Contributors Kanban automation moved this from In progress to Review May 4, 2019
@jakebolam jakebolam changed the title Url fix docs: url fix May 4, 2019
@Berkmann18 Berkmann18 merged commit 582cc86 into master May 4, 2019
All Contributors Kanban automation moved this from Review to Done May 4, 2019
@Berkmann18 Berkmann18 deleted the url-fix branch May 4, 2019 13:17
@all-contributors-release-bot
Copy link
Member

🎉 This PR is included in version 2.14.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Documentation related issue/PR released
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

4 participants