-
Notifications
You must be signed in to change notification settings - Fork 2k
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: Use HEAD rather than master for GitHub URIs #43993
Docs: Use HEAD rather than master for GitHub URIs #43993
Conversation
@@ -7,6 +7,6 @@ Depending on the use case, this may be appropriate. | |||
|
|||
Further reading: | |||
|
|||
https://github.com/Modernizr/Modernizr/blob/master/feature-detects/touchevents.js | |||
https://github.com/Modernizr/Modernizr/blob/HEAD/feature-detects/touchevents.js |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This link is fully broken even with blob/master. Looks like a casualty of a refactor. We should use perma links!
@@ -4,7 +4,7 @@ | |||
* | |||
* @copyright Modernizr © 2009-2015. | |||
* @license See CREDITS.md. | |||
* @see https://github.com/Modernizr/Modernizr/blob/master/feature-detects/touchevents.js | |||
* @see https://github.com/Modernizr/Modernizr/blob/HEAD/feature-detects/touchevents.js |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Ditto
This PR does not affect the size of JS and CSS bundles shipped to the user's browser. Generated by performance advisor bot at iscalypsofastyet.com. |
Caution: This PR affects files in the FSE Plugin on WordPress.com D46104-code has been created so you can easily test it on your sandbox. See this FieldGuide page about developing in the FSE Plugin for more info: PCYsg-ly5-p2 |
8fd816e
to
cd30ce3
Compare
Changes proposed in this Pull Request
blob/master
to useblob/HEAD
which is a default, branch-name-agnostic alias for the "main" branch of a repository, regardless of what it is actually called.Note: Sometimes this alias does not exist if you do some funky stuff but it is the neutral default when you create a new repository through GitHub's UI. When the alias does not exist, the only thing we can do is wait for the project to change their default branch name, if they ever do. I manually checked each repository we reference and didn't find anything other than truly dead links (to files that no longer exist).
Testing instructions
Part of #43395