nvf icon indicating copy to clipboard operation
nvf copied to clipboard

languages/formatters: allow multiple formatters

Open horriblename opened this issue 3 months ago • 4 comments

Allow multiple formatters in vim.languages.<lang>.format.type, deprecates use of non list type

Also deprecates non-list values in vim.languages.*.lsp.server

Sanity Checking

  • [x] I have updated the changelog as per my changes
  • [x] I have tested, and self-reviewed my code
  • [x] My changes fit guidelines found in hacking nvf
  • Style and consistency
    • [x] I ran Alejandra to format my code (nix fmt)
    • [x] My code conforms to the editorconfig configuration of the project
    • [x] My changes are consistent with the rest of the codebase
  • If new changes are particularly complex:
    • [ ] My code includes comments in particularly complex areas
    • [ ] I have added a section in the manual
    • [ ] (For breaking changes) I have included a migration guide
  • Package(s) built:
    • [x] .#nix (default package)
    • [ ] .#maximal
    • [ ] .#docs-html (manual, must build)
    • [ ] .#docs-linkcheck (optional, please build if adding links)
  • Tested on platform(s)
    • [x] x86_64-linux
    • [ ] aarch64-linux
    • [ ] x86_64-darwin
    • [ ] aarch64-darwin

Add a :+1: reaction to pull requests you find important.

horriblename avatar Aug 23 '25 11:08 horriblename

:rocket: Live preview deployed from d5da1a14c3895be269653d47b320697b1d2dafae

View it here:

Debug Information

Triggered by: horriblename

HEAD at: python-multi-formatter

Reruns: 1575

github-actions[bot] avatar Aug 23 '25 11:08 github-actions[bot]

I think this will be easier if you cover all at once, because we will need to cover it before 0.8 anyway.

NotAShelf avatar Aug 23 '25 12:08 NotAShelf

aight bet

horriblename avatar Aug 23 '25 12:08 horriblename

The functions with "deprecated" in their names are bothering me a little. Are we doing to get rid of those? If so, why not do that now since 0.8 is the big bad breaking change.

yea we're getting rid of them in the future, I just put up a little warning for now if you use the old syntax and convert it for you, just cuz I can and also cuz I hate breaking changes that don't say anything useful in logs/warning

horriblename avatar Oct 15 '25 03:10 horriblename

...there are conflicts

it's mergover

NotAShelf avatar Nov 16 '25 18:11 NotAShelf

:trollface:

horriblename avatar Nov 16 '25 18:11 horriblename

✅ Preview has been deleted successfully!

github-actions[bot] avatar Nov 16 '25 18:11 github-actions[bot]