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

Future of this plugin with null-ls being archived? #82

Closed
chrisgrieser opened this issue Aug 5, 2023 · 14 comments · Fixed by #85
Closed

Future of this plugin with null-ls being archived? #82

chrisgrieser opened this issue Aug 5, 2023 · 14 comments · Fixed by #85
Labels

Comments

@chrisgrieser
Copy link

With null-ls being archived in the near future, and up until now new maintainer in sight, I wanted to ask what the future of this plugin is going to be?

I personally work with three different devices, and I find the ensure_installed feature of this plugin immensely useful to ensure I have formatters/linters etc. installed at all devices.

Will this plugin continue ensuring installation of linters/formatters, even without null-ls? Right now, null-ls is still listed as a requirement for this plugin, but technically, null-ls isn't really needed, since this plugin mainly interfaces with mason to install stuff, right? 🙏

@rickypang0219
Copy link

I am also interested and want to contribute in this post. I'm thinking the same way, the possibility of separating the dependency of Null-ls.nvim? Currently I am not familiar the in-dpeth relation between Mason-null-ls and Null-ls My intuition is that installing formatters is closely related to the Mason side rather than Null-ls thus it seems possible.

@chrisgrieser
Copy link
Author

for anyone else finding this issue: I found this plugin, which seems to enable auto-installation as well, and as far as I can tell, it works without null-ls:
https://github.com/WhoIsSethDaniel/mason-tool-installer.nvim

@github-actions
Copy link
Contributor

github-actions bot commented Sep 4, 2023

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 github-actions bot added the Stale label Sep 4, 2023
@github-actions
Copy link
Contributor

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.

@github-actions github-actions bot closed this as not planned Won't fix, can't repro, duplicate, stale Sep 10, 2023
@catgoose
Copy link

catgoose commented Oct 1, 2023

I really like this plugin and I'm wondering if there would be any support for none-ls: https://github.com/nvimtools/none-ls.nvim

@jay-babu
Copy link
Owner

jay-babu commented Oct 1, 2023

I really like this plugin and I'm wondering if there would be any support for none-ls: https://github.com/nvimtools/none-ls.nvim

What changes would be needed to support it? I looked at the docs and I don't see anything that needs to change

@jay-babu jay-babu reopened this Oct 1, 2023
@github-actions github-actions bot removed the Stale label Oct 2, 2023
@xfdj
Copy link

xfdj commented Oct 2, 2023

I really like this plugin and I'm wondering if there would be any support for none-ls: https://github.com/nvimtools/none-ls.nvim

    {
        "jay-babu/mason-null-ls.nvim",
        dependencies = {
            "williamboman/mason.nvim",
            "nvimtools/none-ls.nvim",
        },
    -- ...

So far, directly replacing the null-ls dependency with none-ls is feasible.

@jay-babu
Copy link
Owner

jay-babu commented Oct 3, 2023

for anyone else finding this issue: I found this plugin, which seems to enable auto-installation as well, and as far as I can tell, it works without null-ls: https://github.com/WhoIsSethDaniel/mason-tool-installer.nvim

you can use mason-null-ls for this functionality too. just copy the list you give to the tool-installer to mason-null-ls and it'll work

Repository owner deleted a comment from xfdj Oct 3, 2023
@Zeioth
Copy link

Zeioth commented Oct 15, 2023

I can confirm it works as expected with none-ls

@github-actions
Copy link
Contributor

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 github-actions bot added the Stale label Oct 31, 2023
@daUnknownCoder
Copy link

Bro, Just Replace the NullLsInstall with NoneLsInstall in the commands, and this plugin is good to go, keepin the name of the repo as mason-null-ls gives the original content antique vibes of null-ls

@github-actions github-actions bot removed the Stale label Nov 1, 2023
@jay-babu
Copy link
Owner

Bro, Just Replace the NullLsInstall with NoneLsInstall in the commands, and this plugin is good to go, keepin the name of the repo as mason-null-ls gives the original content antique vibes of null-ls

#85, let me know if this gives modern vibes

Copy link
Contributor

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 github-actions bot added the Stale label Nov 26, 2023
@xfdj
Copy link

xfdj commented Nov 26, 2023 via email

@jay-babu jay-babu linked a pull request Nov 26, 2023 that will close this issue
@jay-babu jay-babu pinned this issue Dec 11, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

7 participants