Skip to content
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

Is this project dead? How can we help? #26

Open
BretFisher opened this issue Jun 20, 2018 · 7 comments
Open

Is this project dead? How can we help? #26

BretFisher opened this issue Jun 20, 2018 · 7 comments

Comments

@BretFisher
Copy link

Hey @samneirinck, I know a lot of docker users that would love this updated to current CLI commands and sub-commands, options, etc.

Maybe we Docker Captains can help spread the word and get you some help with fixes and new features, but I have two questions first:

  1. Are you still willing to lead this repo by accepting PR's, commenting on issues, etc? It would be great if we could use the same PowerShell Module name and repo.

  2. Can you describe a list of issues or needed features to get it working with old and new command format and options?

CC: @StefanScherer

@StefanScherer
Copy link

Thanks Captain @BretFisher ;-)
I encourage my team to use WSL right now, but for the community the posh-docker is still very useful to have same tab expansion experience like in bash.

@samneirinck
Copy link
Owner

Thanks for bringing this to my attention. It's true the project has been pretty silent for quite a while, mainly due to the fact I've switched to bash+WSL to interact with Docker/Kubernetes.

I've tried a few times to introduce support for the management commands, however the implementation was not so trivial, based on the regex approach I used for the tab completion. Other tab completions for docker, like bash, fish, ... use a more extensive system, pretty much handling each command separately. This allows them to clearly identify what should be completed in any scenario. I've done some initial prototyping on this in the pester-tests branch.

To answer your 2 questions:

  1. I can accept PR's, comment on issues, etc, but I'm also willing to give some other people the permissions to do the same. Since I'm no longer using it myself, the project is probably better off with some active maintainers, who are dedicated to support the latest features or bug fixes.
  2. With regards to a list of issues, I think the main issue is to support management commands. However, as noted before, it might involve some serious rework of the module (no longer based on only regex).

Let me know what you all think would be the best way to proceed.

@matt9ucci
Copy link

@BretFisher @StefanScherer @samneirinck
Here's my completion module "DockerCompletion" working for the management commands. It is customizable and not based on the regex approach (See this and this).

Note: It requires PowerShell >= 5.0 and docker cli >= 1.13

@BretFisher
Copy link
Author

@matt9ucci This looks great, do you consider it a replacement for posh-docker? (superset). It looks like you've had it awhile, I'll give it a shot!

@matt9ucci
Copy link

matt9ucci commented Jun 24, 2018

@BretFisher Yes. The DockerCompletion works roughly the same as the bash completion for docker. I have started to develop it last year. The latest version supports Docker v18.03.0. The next release will support v18.06.0. Feedback is welcomed!

I think posh-docker is good for the users who use PowerShell v4.0 or lower and Docker v1.12 or lower. But for the users who use PowerShell v5.0+, including PowerShell Core, and Docker v1.13+, the DockerCompletion should be useful.

@samneirinck
Copy link
Owner

Perhaps we can just add a note to the READNE here then to point towards @matt9ucci's excellent work?

@samneirinck
Copy link
Owner

I've updated the README to clarify the current status, and point to the DockerCompletion github page.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants