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

Some improvements for this Extensions #70

Open
wants to merge 5 commits into
base: master
Choose a base branch
from

Conversation

nicumicle
Copy link

I've added some improvements to this extension:

  • Add a new option for "Stop and Remove": Sometimes, I want to remove a container that is already started. So, instead of clicking on stop and then on remove, we have this new option "stop and remove";
  • Sort container names: When there are a lot of containers, it is kind of hard to find the one you want. I think if the container names are sorted, it is easier to find them;
  • Add --force when removing a container;
  • Display gray color when the status is undefined;

src/docker.js Outdated Show resolved Hide resolved
@nicumicle
Copy link
Author

Hello @gpouilloux ,

I did the changes you requested.

Please have another look at this PR.

@nicumicle nicumicle requested a review from gpouilloux October 12, 2022 04:59
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

Successfully merging this pull request may close these issues.

2 participants