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

Make markdown-lint configurable #3017

Closed
Rokt33r opened this issue May 22, 2019 · 9 comments
Closed

Make markdown-lint configurable #3017

Rokt33r opened this issue May 22, 2019 · 9 comments
Labels
feature request 🌟 Issue is a new feature request.

Comments

@Rokt33r
Copy link
Member

Rokt33r commented May 22, 2019

Issuehunt badges

Current behavior

I cannot opt out markdown-lint.

Expected behavior

I can opt out markdown-lint.

IssueHunt Summary

Sponsors (Total: $0.00)

Become a sponsor now!

Or submit a pull request to get the deposits!

Tips

@Rokt33r Rokt33r added the feature request 🌟 Issue is a new feature request. label May 22, 2019
@issuehunt-oss issuehunt-oss bot added the funded on issuehunt 💵 Issue has received funding that will be rewarded to the contributor solving this issue. label May 22, 2019
@IssueHuntBot
Copy link

@BoostIO has funded $30.00 to this issue.


@discreet
Copy link

The linter should either be turned off entirely or at the very least allowed to be disabled. It's very wrong and very annoying. The fact that I want to jump heading sizes from heading to heading should not be a warning.

@roottool
Copy link
Contributor

Sorry for the inconvenience in my implementation.
I am working on #2983 to edit markdownlint config.
A markdownlint config editor will be implemented in Boostnote config if #2983 will merge.
The default value in a markdownlint config editor is the below.

{
    "default": true
}

You have to change from true to false when you want to turn off markdownlint.

@Flexo013
Copy link
Contributor

For extra usability one should be able to right-click the yellow triangles and have the option to disable only that specific inspection/rule.

Maybe this should be its own separate issue or feature request. It is however dependent on this issue, as configurability is needed for such a feature.

@Rokt33r
Copy link
Member Author

Rokt33r commented May 23, 2019

I'll review #2983 soon!

@issuehunt-oss issuehunt-oss bot removed the funded on issuehunt 💵 Issue has received funding that will be rewarded to the contributor solving this issue. label May 26, 2019
@IssueHuntBot
Copy link

@Rokt33r has cancelled @BoostIO's funding for this issue.(Cancelled amount: $30.00) See it on IssueHunt

@issuehunt-oss issuehunt-oss bot added the funded on issuehunt 💵 Issue has received funding that will be rewarded to the contributor solving this issue. label May 26, 2019
@IssueHuntBot
Copy link

@BoostIO has funded $20.00 to this issue.


@Rokt33r
Copy link
Member Author

Rokt33r commented May 26, 2019

Fixed #3034

@issuehunt-oss issuehunt-oss bot removed the funded on issuehunt 💵 Issue has received funding that will be rewarded to the contributor solving this issue. label May 26, 2019
@IssueHuntBot
Copy link

@Rokt33r has cancelled @BoostIO's funding for this issue.(Cancelled amount: $20.00) See it on IssueHunt

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
feature request 🌟 Issue is a new feature request.
Projects
None yet
Development

No branches or pull requests

5 participants