Skip to content

Lowercase package paths before generating links

Nataliia Radina requested to merge 367656-fix-microsoft-links into master

What does this MR do and why?

Just noticed while testing #367656 (closed) on production that tags to Microsoft and other packages with uppercase letters won't work. Happen to be because of capital letter, so now fixed via lowercasing package paths.

Btw this was not a regression, the links to Microsoft did not work also in .haml version of blobs.

Screenshots or screen recordings

Before After
Screenshot_2022-12-01_at_11.54.45 Screenshot_2022-12-01_at_11.55.07

How to set up and validate locally

Numbered steps to set up and validate the change are strongly suggested.

  1. Add to go.sum file to any local project
  2. Check the tag links which contain capital letters

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Related to #367656 (closed)

Merge request reports

Loading