Skip to content

Migrate nvim <table> deprecations to <params>#3047

Merged
olimorris merged 2 commits intoolimorris:mainfrom
HendrikPetertje:fix-deprecations
Apr 29, 2026
Merged

Migrate nvim <table> deprecations to <params>#3047
olimorris merged 2 commits intoolimorris:mainfrom
HendrikPetertje:fix-deprecations

Conversation

@HendrikPetertje
Copy link
Copy Markdown
Contributor

@HendrikPetertje HendrikPetertje commented Apr 19, 2026

Description

Neovim (nightly builds) currently throw the following error in checkhealth:


- ⚠️ WARNING vim.validate{<table>} is deprecated. Feature will be removed in Nvim 1.0
  - ADVICE:
    - use vim.validate(<params>) instead.
    - stack traceback:
        /Users/peva/.local/share/nvim/lazy/codecompanion.nvim/lua/codecompanion/utils/log.lua:183
        /Users/peva/.local/share/nvim/lazy/codecompanion.nvim/lua/codecompanion/utils/log.lua:293
        =[C]:-1
        /Users/peva/.local/share/nvim/lazy/codecompanion.nvim/lua/codecompanion/utils/ui.lua:2
        =[C]:-1
        /Users/peva/.local/share/nvim/lazy/codecompanion.nvim/lua/codecompanion/config.lua:2
        =[C]:-1
        /Users/peva/.local/share/nvim/lazy/codecompanion.nvim/lua/codecompanion/init.lua:2
        =[C]:-1
        /Users/peva/.config/nvim/lua/plugins/code-companion.lua:38
        /Users/peva/.local/share/nvim/lazy/lazy.nvim/lua/lazy/core/loader.lua:380
        =[C]:-1
        /Users/peva/.local/share/nvim/lazy/lazy.nvim/lua/lazy/core/util.lua:135
        /Users/peva/.local/share/nvim/lazy/lazy.nvim/lua/lazy/core/loader.lua:395
        /Users/peva/.local/share/nvim/lazy/lazy.nvim/lua/lazy/core/loader.lua:362
        /Users/peva/.local/share/nvim/lazy/lazy.nvim/lua/lazy/core/loader.lua:197
        /Users/peva/.local/share/nvim/lazy/lazy.nvim/lua/lazy/core/loader.lua:127
        /Users/peva/.local/share/nvim/lazy/lazy.nvim/lua/lazy/init.lua:112
        /Users/peva/.config/nvim/lua/config/lazy.lua:20
        =[C]:-1
        /Users/peva/.config/nvim/init.lua:18
    - stack traceback:

This change addresses these problems by changing the supplied table to just a single call. (table to params basically)

AI Usage

None.

Related Issue(s)

no issues reported that i can see

Screenshots

After the patch:
image

deprecation warning is gone.

Checklist

  • I've read the contributing guidelines and have adhered to them in this PR
  • I confirm that this PR has been majority created by me, and not AI (unless stated in the "AI Usage" section above)
  • I've run make all to ensure docs are generated, tests pass and StyLua has formatted the code
  • (optional) I've added test coverage for this fix/feature
  • (optional) I've updated the README and/or relevant docs pages

last 2 were not needed

@olimorris olimorris merged commit 23532cf into olimorris:main Apr 29, 2026
6 checks passed
@olimorris
Copy link
Copy Markdown
Owner

Amazing, thanks for this. Nice to finally remove this from my todo list 😄

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