-
-
Notifications
You must be signed in to change notification settings - Fork 180
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
Add context menu to disable rule for line or file scope #224
Comments
I like what you propose better, actually, and added your suggestions into that issue for tracking. I never felt like I had a good plan for how/where to enable globally, so using an approach like yours or using local file overrides seems pretty appealing right now. Actually, I guess I will leave the two issues separate for now, but what I'm guessing will happen is that I implement this one and decline that one. |
I didn't dig through the other repo very far, but I figure there might be some code to get inspiration from |
I'll look at their documentation for how they've approached this, but I don't look at code for other projects to avoid the risk of licensing, etc. issues. |
I've thought about how to do this for line scope and the challenge is that problems quickly arise with how/where to add the suppression comment: Adding `Code with space ` and hard break <!-- markdownlint-disable-line no-space-in-code -->
Text text text Using ```code
<!-- markdownlint-disable-next-line blanks-around-fences -->
```
Text text text Using Using
|
Personally, I'd probably use the |
Like what https://github.com/microsoft/vscode-eslint does in that it allows for disabling through the inline comments at either the file or line scope, when you don't want to globally disable a rule (like asked in #210)
The text was updated successfully, but these errors were encountered: