docs(datasource/deb): All items in the urls array are wrapped between []() character.

[Prtik12]

Changes

Context

• Replace bare links in index.ts files in the modules/versioning/*/ pattern with good links #30312

Documentation (please check one with an [x])

• I have updated the documentation, or
• No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

• Code inspection only, or
• Newly added/modified unit tests, or
• No unit tests but ran on a real repository, or
• Both unit tests + ran on a real repository

[CLAassistant]

All committers have signed the CLA.

[rarkins]

Does this already work if merged, or needs some changes to our docs generation first?

[HonkingGoose]

Fails if merged

Does this already work if merged, or needs some changes to our docs generation first?

The mkdocs build step fails:

WARNING -  Doc file 'modules/versioning/deb/index.md' contains a link '[Debian Policy Manual - Version Control Fields](https://www.debian.org/doc/debian-policy/ch-controlfields.html#version)', but the target 'modules/versioning/deb/[Debian Policy Manual - Version Control Fields](https:/www.debian.org/doc/debian-policy/ch-controlfields.html' is not found among documentation files.
WARNING -  Doc file 'modules/versioning/deb/index.md' contains a link '[Debian Manual - deb-version(7)](https://manpages.debian.org/unstable/dpkg-dev/deb-version.7.en.html)', but the target 'modules/versioning/deb/[Debian Manual - deb-version(7)](https:/manpages.debian.org/unstable/dpkg-dev/deb-version.7.en.html)' is not found among documentation files.

This is the code that generates the ## References section and puts the links in a list:

renovate/tools/docs/utils.ts

Lines 51 to 58 in 9b64288

	export function formatUrls(urls: string[] | null | undefined): string {
	if (Array.isArray(urls) && urls.length) {
	return `## References\n\n${urls
	.map((url) => ` - [${url}](${url})`)
	.join('\n')}\n\n`;
	}
	return '';
	}

I'll let you programmers figure out what the proper fix is. 😉

Scope

I guess this PR only tries to fix the bad links in the Debian datasource file, but there are still bare links in other files, for example:

renovate/lib/modules/versioning/composer/index.ts

Lines 12 to 17 in 9b64288

	export const urls = [
	'https://getcomposer.org/doc/articles/versions.md',
	'https://packagist.org/packages/composer/semver',
	'https://madewithlove.be/tilde-and-caret-constraints/',
	'https://semver.mwl.be',
	];

Test that finds bad links

I would also like to see a test to check for bad links in the const urls. That way we can use the test to find all bad links. 😄

[HonkingGoose]

See my comment in the PR for more info, to summarize:

• The docs build should pass
• The docs should have a list with human-readable links