Skip to content
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

XS✔ ◾ Markdown Link Check: Skipping Email Address #557

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

muiriswoulfe
Copy link
Member

@muiriswoulfe muiriswoulfe commented Nov 11, 2024

Summary

This updates the .github/linters/markdown-link-check.json file to add a new exclusion pattern for mailto links. This ensures that the linter does not validate email addresses, where validation will now always fail.

Testing

Test Types

  • Unit tests
  • Manual tests

Unit Test Coverage

100%

@muiriswoulfe muiriswoulfe added the bug Something isn't working label Nov 11, 2024
@muiriswoulfe muiriswoulfe self-assigned this Nov 11, 2024
@muiriswoulfe muiriswoulfe requested a review from a team as a code owner November 11, 2024 12:07
Copy link
Contributor

PR Metrics

Thanks for keeping your pull request small.
Thanks for adding tests.

Lines
Product Code -
Test Code -
Subtotal -
Ignored Code 3
Total 3

Metrics computed by PR Metrics. Add it to your Azure DevOps and GitHub PRs!

@github-actions github-actions bot changed the title Markdown Link Check: Skipping Email Address XS✔ ◾ Markdown Link Check: Skipping Email Address Nov 11, 2024
@muiriswoulfe muiriswoulfe enabled auto-merge (squash) November 11, 2024 12:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant