-
Notifications
You must be signed in to change notification settings - Fork 25
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
recommendation: put nerd-icons-dired
and nerd-icons-completion
in this repo
#8
Comments
nerd-icons-dired
and nerd-icons-completion
in this branchnerd-icons-dired
and nerd-icons-completion
in this repo
Actually this makes sense. I'll consider this. :) |
Pros for this consideration: since this is a new project, we want this project gain popularity as quick as possible. Putting essential features into separate repos is definitely not a good decision since I can bet 99% of users want to display icons in minibuffer and dired and they need to visit multiple repos and install many packages for one thing. |
Also for I don't think it's a big problem, whatever they are merged into one repo. Same situation for |
I've also created a simple plugin for |
It seems silly to install a new package for each tiny feature where the Readme plus Cask is about the same size as the package itself. |
Counter argument: this would make it harder (impossible?) to communicate API breaking changes via git tags and semantic versioning at the level of granularity of those extensions. |
minibuffer completion
anddired
are emacs builtin features. I see no reasons why we don't want to put those two extensions into this repo. They can be separated files, but including them in this repo makes totally sense.It's like
consult
has included many extensions likeconsult-buffer
,consult-ripgrep
stuffs in theconsult
repo.The text was updated successfully, but these errors were encountered: