Skip to content

feat!: set no-missing-link-fragments default ignoreCase to true #447

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

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

Conversation

TKDev7
Copy link
Contributor

@TKDev7 TKDev7 commented Jun 26, 2025

Prerequisites checklist

What is the purpose of this pull request?

This PR updates the default behavior of the no-missing-link-fragments rule to use ignoreCase: true.

What changes did you make? (Give an overview)

  • Set the default value of the ignoreCase option to true in the no-missing-link-fragments rule.
  • Updated the documentation to reflect the new default.
  • Adjusted and added tests to ensure the rule works as expected with the new default setting.

Related Issues

Fixes #441

Is there anything you'd like reviewers to focus on?

@TKDev7 TKDev7 marked this pull request as draft June 26, 2025 12:36
@TKDev7 TKDev7 marked this pull request as ready for review June 27, 2025 06:20
@github-project-automation github-project-automation bot moved this to Needs Triage in Triage Jun 28, 2025
@lumirlumir lumirlumir moved this from Needs Triage to Implementing in Triage Jun 28, 2025
lumirlumir
lumirlumir previously approved these changes Jun 29, 2025
Copy link
Member

@lumirlumir lumirlumir left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM, I’ve left a comment about a minor typo.

Would like @snitin315 to verify before merging.

@lumirlumir lumirlumir moved this from Implementing to Second Review Needed in Triage Jun 29, 2025
@lumirlumir lumirlumir requested a review from snitin315 June 29, 2025 06:25
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Second Review Needed
Development

Successfully merging this pull request may close these issues.

Change Request: Make no-missing-link-fragments default ignoreCase: true
3 participants