click-contrib / click-completion

Add or enhance bash, fish, zsh and powershell completion in Click
MIT License
286 stars 32 forks source link

Clarify whether this repo is deprecated or not #41

Open zbentley opened 3 years ago

zbentley commented 3 years ago

This repo appears to duplicate (and maybe pre-date?) the official Click completion functionality.

Is this the predecessor to that/was that rolled into Click core at some point? If so, it might be good to:

If this is not a dupe of the click core functionality, and is instead an alternative implementation (or internal implementation component of the Click core func), it would be good to indicate that prominently in the README.

tmax22 commented 5 months ago

yup. probably dead

n1ngu commented 5 months ago

I understand part of the answer is here https://github.com/click-contrib/click-completion/issues/37

Click's completion system has been rewritten [...] Due to all these changes, the click-completion won't work with the next release of Click. We've talked about merging this project into Click before, but I think we pretty much have what we want in core at this point. One option is to make a release that specifies click<8 in install_requires, then have the next version specify click>=8 and continue to support the parts that Click doesn't provide.

So

But this is still to be done and the project feels very stale. I guess the release of click 8 removed most of the interest on maintaining click-completion for both the maintainers and the community.

So: ping @glehmann @Konubinix , sorry to bother you. A word about this? IMHO one of the following should happen

Thanks!

Konubinix commented 5 months ago

To me, it is dead. In my projects, I try to use the native completion as much as possible and suggest you do the same.

I agree with your suggestions. I leave to @glehmann to make actions he deems appropriate.

glehmann commented 5 months ago

click's native completion should be used with click 8.0. This one is for click before 8.0.

If click's completion is not as complete as this one, we (as a community) should work on the missing features directly in click.

I'll add a notice at the top of the README and update the required click version to < 8.0

a-detiste commented 2 months ago

please tag a maintenance release