feat: add option to disable context menu entry#374
Conversation
|
I can help review this one as well. Could you rebase, add a bit more context and relevant tests (if applicable)? |
2fcbd24 to
489b86a
Compare
|
editorconfig extension adds this context menu entry: I've:
|
|
👍 Do you know if this feature is testable? This sounds like a useful feature that we should better ensure it survives future changes. |
|
I don't think it's possible to check whether the entry actually appears in the menu or not, but the state can be tested. I will add tests later. |
4fb01db to
c80343d
Compare
|
I've removed the code related to hiding the entry when an |
xuhdev
left a comment
There was a problem hiding this comment.
LGTM
This will go to 0.17.0, which I plan to release shortly after 0.16.7 (allow a little time to track down some bugs as there was a large dependency change in 0.16.7)

and hide the entry when editorconfig already existstscw/o errors (same asnpm run build).npm run lintw/o errors.fixes #331
fixes #212