search

jay-babu / mason-null-ls.nvim

GNU Affero General Public License v3.0
549 stars 22 forks source link

Why are not all sources supported? #70

Closed nieomylnieja closed 1 year ago

nieomylnieja commented 1 year ago

I see that lua/mason-null-ls/mappings/filetype.lua lists all the null-ls supported sources but some of these are commented out, why is that, how do you auto generate this file? I also see that lua/mason-null-ls/mappings/source.lua has some custom mappings which don't simply adhere to - and _ substitution, the thing I need in my project is ansiblelint which for Mason is ansible-lint, and it currently does not work with ensure_installed. I'd be happy to help out, just want to know if this is intended, todo or missed? Thanks!

nieomylnieja commented 1 year ago

To clarify, passing ansible-lint (the Mason way) to ensure_installed does work, but then this is not really a bridge, just an install utility :)

jay-babu commented 1 year ago

I see that lua/mason-null-ls/mappings/filetype.lua lists all the null-ls supported sources but some of these are commented out, why is that, how do you auto generate this file?

this file wasn't autogenerated. a contributor had created it. at the time, not many things existed in mason, hence less things are supported.

I need in my project is ansiblelint which for Mason is ansible-lint, and it currently does not work with ensure_installed.

feel free to add with a PR

nieomylnieja commented 1 year ago

It had "-- THIS FILE SHOULD BE GENERATED" hanging at the top, so this was more of a TODO then the current state? :P

I'll create the PR ;)

nieomylnieja commented 1 year ago

Also, some of the things listed there are actually parts of other tools, like terraform_fmt is in reality part of terraformls, or pycodestyle being part of flake8.

github-actions[bot] commented 1 year ago

This issue has been automatically marked as stale. If this issue is still affecting you, please leave any comment (for example, "bump"), and we'll keep it open. If you have any new additional information—in particular, if this is still reproducible in the latest version of Electron or in the beta—please include it with your comment!

github-actions[bot] commented 1 year ago

This issue has been closed due to inactivity, and will not be monitored. If this is a bug and you can reproduce this issue on a supported version of Electron please open a new issue and include instructions for reproducing the issue.