1564815 - GeckoView's CHANGELOG has incorrect links due to reused link references

Status: RESOLVED FIXED

(GeckoView :: General, enhancement, P3)

Description

[Rob Wu [:robwu]]

Every link in GeckoView's CHANGELOG.md should have a unique reference (i.e. [link here][reference here]). This is not the case; there are four instances of reused references, resulting in links that contain the wrong target URL.

$ wget https://hg.mozilla.org/mozilla-central/raw-file/tip/mobile/android/geckoview/src/main/java/org/mozilla/geckoview/doc-files/CHANGELOG.md
$ grep -oP '^\[[0-9.]+' CHANGELOG.md | sort | uniq -c | sort -n
...
      2 [67.2
      2 [68.23
      2 [69.2
      2 [69.3

• 67.2 - :69 and :142
• 68.23 - :115 and :120
• 69.2 - :16 and :27
• 69.3 - :21 and :28

Comment 1

[Chris Peterson [:cpeterson]]

Is this a merge issue? We should also linkify bug numbers in CHANGELOG.

Comment 2

[Emily Toop (:fluffyemily)]

Attachment: Fix broken CHANGELOG links.

...em. 5. Add links to changelog entries that did not have them, but should have

Comment 3

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/6ad8ff235dc4
Fix broken CHANGELOG links.  r=agi

Comment 4

[Andrei Ciure[:aciure]]

https://hg.mozilla.org/mozilla-central/rev/6ad8ff235dc4

Comment 5

[Jan Henning [:JanH]]

Hmm, removing the additional newlines between each change log entry didn't really improve the readability when looking at the unformatted source file in my opinion...

Comment 6

[Chris Peterson [:cpeterson]]

firefox69=wontfix because we don't need to uplift changelog fixes to Beta.