-
Notifications
You must be signed in to change notification settings - Fork 586
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Update .markdown-link-check.json #2056
Conversation
Ignore links to internal builds from link checker. They are used as documentation for how to release.
@pgrawehr @Ellerbach any clues why is the pattern not working? Is the build picking this up after it's merged? |
This is indeed strange, given that a) that URL is already excluded from checking and b) this is confirmed when I run the test locally:
(the [/] means that the check of that link is skipped.) It seems that the build is completely ignoring the .json file. |
Great. The issue is an update of the mardown-link-checker. If I use the latest version, it behaves the same. |
The latest version (3.11.0) is broken. It completely ignores the configuration file.
@krwq Fixed the issue by nailing to the older version of the markdown-link-checker. |
Issue created here: tcort/markdown-link-check#246 |
Ignore links to internal builds from link checker. They are used as documentation for how to release.
Fixes issue seen in: #2055
Microsoft Reviewers: Open in CodeFlow