feat: enable config for obsidian-md-plugins rules only#70
feat: enable config for obsidian-md-plugins rules only#70chrisgrieser wants to merge 1 commit intoobsidianmd:masterfrom
Conversation
|
Some external rules should still be enabled by default or have documented replacements in the other linter.
|
|
I don't know about enabled by default, because that would exactly result in the problem of duplicate diagnostics with the other linters. Maintaining a list of equivalent rules with other linters is going to be tedious, since there are numerous linters which might also change rule structure over time. I think the best solution would be to just document the list of non-obsidian-but-still-recommended rules in the Readme? |
I looked into this. Most of these rules have a Biome equivalent, but not all.
|
|
|
I've been running into this issue too, thank you for the PR! I don't feel that ESLint provides the quality of linting that I get from TypeScript+Biome, and I'd like to turn off most of the default ESLint stuff, but I can't do that without losing all the specific-to-Obsidian rules which I do really want. In my opinion it would be OK to continue enabling that short list of rules by default, especially if they're a sticking point in review, as things like "no-implied-eval" are quite rare I don't think the occasional duplication of those rare warnings is a major developer experience problem. It would be a great start just to be able to turn off the more basic code quality warnings with no particular relation to Obsidian plugin development. :) |
If you use a linter other than eslint (in my case biome), you may want to only enable Obsidian-specific rules, without all the other eslint rules resulting in duplicate warnings, the need to configure eslint for your specific setup, etc.
currently, there is no direct export of the obsidian-plugin-only rules, so I added that, along with a documentation how to configure eslint for that.