Skip to content

Conversation

@per1234
Copy link
Collaborator

@per1234 per1234 commented Apr 25, 2025

This reverts commit cfd1546.

This configuration entry was previously required in order to work around a bug in the markdown-link-check tool (tcort/markdown-link-check#202) that resulted in false positives for links to anchors created by HTML anchor tags.

The bug in the "markdown-link-check" tool has since been fixed (tcort/markdown-link-check#331), and this project updated to use the version of the tool with that fix (#587). So this configuration entry is no longer required.

The now superfluous configuration entry is hereby removed in order to ensure more comprehensive link check coverage, and avoid unnecessary clutter in the "markdown-link-check" configuration file.

This reverts commit cfd1546.

This configuration entry was previously required in order to work around a bug in the "markdown-link-check" tool that
resulted in false positives for links to anchors created by HTML anchor tags.

The bug in the "markdown-link-check" tool has since been fixed, and this project updated to use the version of the tool
with that fix. So this configuration entry is no longer required.

The now superfluous configuration entry is hereby removed in order to ensure more comprehensive link check coverage,
and avoid unnecessary clutter in the "markdown-link-check" configuration file.
@per1234 per1234 added type: imperfection Perceived defect in any part of project topic: infrastructure Related to project infrastructure labels Apr 25, 2025
@per1234 per1234 self-assigned this Apr 25, 2025
@per1234 per1234 merged commit 5dd88ff into arduino:main Apr 25, 2025
13 checks passed
@per1234 per1234 deleted the remove-markdown-link-check-workaround branch April 25, 2025 05:16
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

topic: infrastructure Related to project infrastructure type: imperfection Perceived defect in any part of project

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant