nvim-tree / nvim-web-devicons

lua `fork` of vim-web-devicons for neovim
MIT License
2.02k stars 182 forks source link

Remove `web` from the plugin's name #171

Open T-727 opened 1 year ago

T-727 commented 1 year ago

nvim-web-devicons -> nvim-devicons

Not exactly necessary in any way but it is cleaner and makes more sense as this plugin isn't really web oriented.

b0o commented 1 year ago

I agree. See: https://github.com/ryanoasis/vim-devicons/issues/64

gegoune commented 1 year ago

It definitely makes sense to rename; question is whether it's worth breaking users' configs?

T-727 commented 1 year ago

It definitely makes sense to rename; question is whether it's worth breaking users' configs?

Doesn't necessarily have to, a dummy file and with a deprecation warning for a while should make the transition fairly smooth

vollowx commented 1 year ago

Any progress or decisions for this?

gegoune commented 1 year ago

PR welcome, would need a shim to avoid breaking everyones config.

vollowx commented 1 year ago

What about copying this repo to nvim-tree/nvim-devicons, then add the rename info to README and achieve this repo, then we can simply global remove every -web in nvim-tree/nvim-devicons.

alex-courtis commented 1 year ago

RE plugin name/location, github does some redirection for us. After moving nvim-tree, https://github.com/kyazdani42/nvim-tree.lua redirects to the new location.

Care must still be taken with the plugin itself to ensure that we don't have any assumptions.