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
misc: fix broken links in changelog #14130
Conversation
Use <https://github.com/GoogleChrome/lighthouse/compare/> for "Full Changelog", instead of <https://github.com/compare/>.
Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA). View this failed invocation of the CLA check for more information. For the most up to date status, view the checks section at the bottom of the pull request. |
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.
Good catch, thanks for the PR. I'll go ahead and fix all of the release pages you mentioned.
@takenspc please sign the CLA |
This comment was marked as spam.
This comment was marked as spam.
[Full Changelog]({{~@root.repoUrl}}/compare/{{previousTag}}...{{currentTag}}) | ||
[Full Changelog]( | ||
{{~#if @root.host}}{{~@root.host}}/{{/if~}} | ||
{{~#if @root.owner ~}}{{@root.owner}}/{{/if~}} |
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.
do all these newlines work? yarn changelog
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.
Looked fine when I tried it:
<a name="9.5.0"></a>
# 9.5.0 (2022-06-15)
[Full Changelog](https://github.com/GoogleChrome/lighthouse/compare/v7.3.0...v9.5.0)
~~ TODO: https://chromiumdash.appspot.com/schedule ~~
We expect this release to ship in the DevTools of [Chrome XX](https://chromiumdash.appspot.com/schedule), and to PageSpeed Insights within 2 weeks.
## Notable Changes
~~ TODO: Move notable changes here ~~
## New Audits
* add work-during-interaction diagnostic ([#13982](https://github.com/GoogleChrome/lighthouse/pull/13982))
* add responsiveness metric for timespans ([#13917](https://github.com/GoogleChrome/lighthouse/pull/13917))
* lcp-lazy-loaded ([#12838](https://github.com/GoogleChrome/lighthouse/pull/12838))
* fr: uses-responsive-images-snapshot ([#12714](https://github.com/GoogleChrome/lighthouse/pull/12714))
Summary
"Full Changelog" links of changelogs are broken since v9.3.0.
Expected urls are
https://github.com/GoogleChrome/lighthouse/compare/v1...v2
though actual urls arehttps://github.com/compare/v1..v2
.I guess that updating
conventional-changelog-cli
cause this issue.This PR fixes changelog template to genearete correct urls and broken links of changelog.md.
Related Issues/PRs
This issue seems trivial and fix is straightforward so I skipped creating an issue. Let me know if I should create an issue.