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

Ability to give megalinter container an arbitrary name #1557

Closed
dmitri-trofimov opened this issue Jun 24, 2022 · 1 comment · Fixed by #1558
Closed

Ability to give megalinter container an arbitrary name #1557

dmitri-trofimov opened this issue Jun 24, 2022 · 1 comment · Fixed by #1558
Labels
enhancement New feature or request

Comments

@dmitri-trofimov
Copy link
Contributor

Is your feature request related to a problem? Please describe.
As a megalinter user on Windows I'd like to have an ability to run megalinter and supply a name to the Docker container it runs in so that I have less random container names in my Docker Desktop application.

Describe the solution you'd like
By looking at the megalinter runner code, it seems quite easy to add an option like --containername and pass it to Docker docker ... --name <containername>. I can try and create a PR for that.

Describe alternatives you've considered
I don't know of any alternatives.

Additional context
When a Docker container is run, it is given a random name (redesigned-train or quantum-fish...). Although I am not sure if this happens on Linux machines, it definetly happen on Windows in Docker Desktop. It also saves the containers, and after a few runs you get a list of many random looking containers in the list. I'd much rather prefer havin only one line corresponding to the megalinter.

image

@dmitri-trofimov dmitri-trofimov added the enhancement New feature or request label Jun 24, 2022
@dmitri-trofimov dmitri-trofimov changed the title Give megalinter Docker container an arbitrary name Ability to give megalinter container an arbitrary name Jun 24, 2022
@Kurt-von-Laven
Copy link
Collaborator

Yes, this also happened on Linux, so thank you for the fix!

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

Successfully merging a pull request may close this issue.

2 participants