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

Fix links in documentation #343

Merged
merged 3 commits into from
Jul 23, 2018
Merged

Conversation

noahbrenner
Copy link
Contributor

Just a little polishing:

  • One page was moved to a new URL.
  • A fragment link in the README needed to be updated.
  • Most of the URLs can use https instead of http
  • Links in the contributor covenant didn't match the source and one was unnecessarily specified twice (as the link text and as the href).

If you'd like any adjustments or want these commits squashed, just let me know!

For any URLs that automatically redirect from `http` to `https`, use the
latter version. The only URL that doesn't redirect to `https` was left
unchanged: http://lea.verou.me/2012/01/why-tabs-are-clearly-superior/
(When using `https` for this URL, Chrome shows "Your connection to this
site is not fully secure", so it's probably better to just leave it)
* Use an inline link, which eliminates duplication of the URL.
* Include "www" and full path, eliminating unnecessary redirects.

These lines now match the reference document:
https://www.contributor-covenant.org/version/1/4/code-of-conduct.md
@sindresorhus sindresorhus merged commit 404d81b into xojs:master Jul 23, 2018
@sindresorhus
Copy link
Member

Good cleanup! Thanks :)

@noahbrenner noahbrenner deleted the dead-links branch July 23, 2018 09:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants