romgrk / barbar.nvim

The neovim tabline plugin.
2.23k stars 83 forks source link

v2.0.0 Tracking Issue #394

Open Iron-E opened 1 year ago

Iron-E commented 1 year ago

In #386, there was discussion of breaking compatibility with ^1.0.0 by removing support for g:bufferline. This issue is to track what changes we are looking to make for a potential v2 release.

otavioschwanck commented 1 year ago

i like the changes on setup, i think is worth

romgrk commented 1 year ago

I think we should aim to have a gradual non-breaking migration, until the last step of the process.

1..setup() would accept the exact same format as g:bufferline, we update the doc to use .setup() as the only documented method, but g:bufferline keeps working as before

  1. we add a one-time notification when neovim opens announcing the change in format, with a clear migration path
  2. we remove support, but show an error notification if g:bufferline is present telling users to .setup(vim.g.bufferline) please
  3. remove support & notification

We should also mark our commit as breaking with the conventional commits notation (feat!: ...), I've seen some package managers display breaking notices when they detect those when the user upgrades their plugins.

romgrk commented 1 year ago

I'm cool with the bufferline => barbar rename as well, but let's try to migrate gradually as well.

Iron-E commented 1 year ago

Thanks for the feedback. I'll keep this issue updated the transition can be monitlred item-by-item at a glance

ttytm commented 1 year ago

+1 on the neovim version upper from my side. 0.9 is coming up and supporting for the last two "stable" versions of an early project is fair enough.

Would adding a formatting standard / config that can be automated be something for that version release?

Could be stylua, lua_ls or .editorconfig. Adding any of these would introduce formatting changes. So why not now? Shouldn't be a lot of work either, adding the desired config and running one command on the full project should probably be it.

edit: Just saw there is #411