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

support for "suppress <lint rule>" textDocument/codeAction action #50

Open
MagicDuck opened this issue Mar 29, 2021 · 4 comments
Open

Comments

@MagicDuck
Copy link

Hi there,

I was poking around with this to replace my coc.nvim eslint config, but I noticed that there is no support yet for code action to add stuff like eslint-disable-next-line:

// eslint-disable-next-line import/no-extraneous-dependencies
const glob = require('glob');

This type of code action can be a really handy time saver.

@VVKot
Copy link

VVKot commented Apr 4, 2021

@MagicDuck are you getting any code actions though? I found no references to them in the codebase and didn't see anything in my editor when setting up, say, ESLint.

@iamcco
Copy link
Owner

iamcco commented May 7, 2021

It's general lint server, so it's hard to add codeActions if there is no common format of the code action from the lint.

@MagicDuck
Copy link
Author

ended up with slightly hacky solution in my lsp config: see this: neovim/neovim#14258 (comment)

@mjlbach
Copy link

mjlbach commented May 25, 2021

If anyone wants this feature, there now exists https://github.com/jose-elias-alvarez/null-ls.nvim which allows you to use the language server API with arbitrary neovim code action etc.

Edit: whoops, wrong issue, keeping here for magicduck

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants