Contribuiting file with links outdated

[Rodrigo-Novas]

Is your feature request related to a problem?

It is not a problem but the documentation of the contribution file gave me some problems in understanding the rules that were proposed.

The links used in the contributing.md file had inconsistencies, for example some links were permalinks and others were not.

For cases where a specific line of code is referenced, it is always recommended to use permalinks, this gives us the possibility of having the part of code that we referred to at the exact moment of putting together the documentation.

Below I show you some examples of links that were wrong:

I change

opentelemetry-python-contrib/instrumentation/opentelemetry-instrumentation-requests/src/opentelemetry/instrumentation/requests/__init__.py

Line 111 in 3ec7736

if context.get_value("suppress_instrumentation") or context.get_value(

For

opentelemetry-python-contrib/opentelemetry-instrumentation/src/opentelemetry/instrumentation/utils.py

Line 191 in 2518a4a

with _suppress_instrumentation(_SUPPRESS_INSTRUMENTATION_KEY):

Because the implementation of suppress_instrumentation has been changed

I change

opentelemetry-python-contrib/instrumentation/opentelemetry-instrumentation-flask/src/opentelemetry/instrumentation/flask/__init__.py

Line 91 in 0fcb60d

if not _excluded_urls.url_disabled(flask.request.url):

For

opentelemetry-python-contrib/instrumentation/opentelemetry-instrumentation-flask/src/opentelemetry/instrumentation/flask/__init__.py

Line 327 in 2518a4a

or not excluded_urls.url_disabled(flask.request.url)

Describe the solution you'd like

To amend this, the links should be replaced with permalinks where they refer to a line of code, while the links that refer to files or external content should be left as a link.

Additionally, you should follow the rules that markdown proposes for your md files (I found several style rules that were not followed), a good guide is to use some markdownlint.

Some of the rules that are not followed:

• https://github.com/markdownlint/markdownlint/blob/main/docs/RULES.md#md034---bare-url-used
• https://github.com/markdownlint/markdownlint/blob/main/docs/RULES.md#md012---multiple-consecutive-blank-lines
• https://github.com/markdownlint/markdownlint/blob/main/docs/RULES.md#md032---lists-should-be-surrounded-by-blank-lines
• https://github.com/markdownlint/markdownlint/blob/main/docs/RULES.md#md014---dollar-signs-used-before-commands-without-showing-output
• https://github.com/markdownlint/markdownlint/blob/main/docs/RULES.md#md026---trailing-punctuation-in-header

Additional context

Some useful links:

https://www.markdownguide.org/basic-syntax/

https://github.com/orgs/community/discussions/35615#discussioncomment-3834745

[Rodrigo-Novas]

I created a PR about this #2240