-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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
📖 Move extension markdown template into its own file #25164
Conversation
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.
LGTM overall. Two comments below.
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.
One more comment about file extensions.
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.
Thanks for cleaning up the code. Since this is a fairly substantial refactor, I've added some comments / requested some changes below. Let me know if I'm off on any of them.
Another request: The changes to the link checker are mostly unrelated to the original purpose of this PR. Can you separate those changes into a separate PR? (Doing so after addressing my comments is fine.) |
Also some modernizing and cleanup. (Offshoot of #25164)
…#25226) Also some modernizing and cleanup. (Offshoot of ampproject#25164)
Keeps `${name}` and `${year}` as interpolation tokens.
Keeps
${name}
and${year}
as interpolation tokens.