Skip to content

docs: remove plenary from install dependencies - #448

Merged
sudo-tee merged 1 commit into
sudo-tee:mainfrom
fengjikui:codex/remove-plenary-runtime-dep
Jul 6, 2026
Merged

docs: remove plenary from install dependencies#448
sudo-tee merged 1 commit into
sudo-tee:mainfrom
fengjikui:codex/remove-plenary-runtime-dep

Conversation

@fengjikui

Copy link
Copy Markdown
Contributor

Summary

  • remove nvim-lua/plenary.nvim from the lazy.nvim installation example because the plugin runtime no longer requires it
  • remove the redundant CI step that installed plenary globally; the test init still provisions the test harness under deps/ when tests run

Closes #394 for the user-facing/runtime dependency path while keeping the current test harness unchanged.

Validation

  • git diff --check
  • rg -n "nvim-lua/plenary.nvim|Install plenary|plenary.nvim" README.md .github/workflows/tests.yml lua plugin ftplugin (no matches)
  • Could not run ./run_tests.sh -t minimal locally because this machine does not have nvim installed; the command exits before tests with nvim: command not found.

@sudo-tee

sudo-tee commented Jul 6, 2026

Copy link
Copy Markdown
Owner

Thanks for the PR.

I will have a look to ensure everything is fine. But at first everything seems fine. The deps are indeed only needed and provided by the test harness.

@sudo-tee

sudo-tee commented Jul 6, 2026

Copy link
Copy Markdown
Owner

All seem to work fine in CI and on local.

Thanks for the cleanup

@sudo-tee
sudo-tee merged commit 79e45a0 into sudo-tee:main Jul 6, 2026
5 checks passed
disrupted pushed a commit to disrupted/opencode-native.nvim that referenced this pull request Jul 23, 2026
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.

plenary dependency no longer actively maintained and officially archived soon.

2 participants