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

fix(docs): add missing plenary.nvim plugin #239

Closed
wants to merge 1 commit into from

Conversation

jfly
Copy link
Collaborator

@jfly jfly commented Jan 12, 2025

This has always been a dependency of none-ls, but now it recently became necessary to build our docs
(this file gets imported during autogen, which requires plenary:

local async = require("plenary.async")
)

This has always been a dependency of none-ls, but now it recently became
necessary to build our docs
(this file gets imported during `autogen`, which requires plenary:
https://github.com/nvimtools/none-ls.nvim/blob/0143bb5f117f76863cf0c24bcf0724d3137f04d2/lua/null-ls/builtins/formatting/nix_flake_fmt.lua#L5)
@jfly jfly requested a review from mochaaP January 12, 2025 01:17
@jfly jfly mentioned this pull request Jan 12, 2025
@mochaaP
Copy link
Member

mochaaP commented Jan 12, 2025

I think it's possible to move the require call inside the function to avoid including it when generating docs

@jfly
Copy link
Collaborator Author

jfly commented Jan 12, 2025

I think it's possible to move the require call inside the function to avoid including it when generating docs

What's going to prevent someone from making a similar mistake in the future, though? I'd argue that it's better to set up a CI environment that's more reflective of the reality that our plugin will actually run in.

@mochaaP
Copy link
Member

mochaaP commented Jan 12, 2025

Fixed in 326afeb

@mochaaP mochaaP closed this Jan 12, 2025
@jfly
Copy link
Collaborator Author

jfly commented Jan 12, 2025

Cool, looks like the docs have updated. Thanks =)

@jfly jfly deleted the fix-broken-autogen branch January 12, 2025 07:18
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

Successfully merging this pull request may close these issues.

2 participants