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

feat: allow to override docker command and user #783

Merged
merged 11 commits into from Oct 16, 2023

Conversation

CedricCabessa
Copy link
Contributor

Add an option to specify a docker command and a docker user.

It is useful if you need to customize your image before running renovate.

It maybe a partial option for
renovatebot/renovate#8804

The idea start from this discussion
renovatebot/renovate#23500

Add an option to specify a docker command and a docker user.

It is useful if you need to customize your image before running `renovate`.

It maybe a partial option for
renovatebot/renovate#8804

The idea start from this discussion
renovatebot/renovate#23500
Copy link
Member

@viceice viceice left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

don't like it, but ok

@viceice viceice requested a review from jdbruijn July 21, 2023 18:36
@viceice
Copy link
Member

viceice commented Jul 21, 2023

please add an e2e test

@CedricCabessa
Copy link
Contributor Author

I added a test (actually modifying the existing one to use the new feature).

You say "you don't like it" :-), happy to hear your thought about it, it was a way to solve my own problem but if you see a better solution let me know

@CedricCabessa
Copy link
Contributor Author

hey @viceice just a quick heads up
Can you consider a merge? Or let me know if you want to see something changed

example/entrypoint.sh Outdated Show resolved Hide resolved
src/renovate.ts Outdated Show resolved Hide resolved
CedricCabessa and others added 2 commits August 7, 2023 09:06
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
@CedricCabessa
Copy link
Contributor Author

@viceice sorry for the delay, I was off 🌴

@CedricCabessa
Copy link
Contributor Author

@viceice 👋 ;-)

.github/workflows/build.yml Show resolved Hide resolved
README.md Outdated
Comment on lines 19 to 20
- [`docker-cmd-file`](#docker-cmd-file)
- [`docker-user`](#docker-user)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We should sort aphabetically

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

done

README.md Outdated Show resolved Hide resolved
CedricCabessa and others added 2 commits August 25, 2023 11:01
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
@CedricCabessa
Copy link
Contributor Author

Hey @viceice, any chance to have this merged & released?

@viceice
Copy link
Member

viceice commented Sep 14, 2023

Hey @viceice, any chance to have this merged & released?

use the re-request review button

src/renovate.ts Outdated Show resolved Hide resolved
src/renovate.ts Outdated Show resolved Hide resolved
src/renovate.ts Outdated Show resolved Hide resolved
CedricCabessa and others added 2 commits October 16, 2023 10:49
Co-authored-by: Michael Kriese <michael.kriese@visualon.de>
@viceice viceice enabled auto-merge (squash) October 16, 2023 09:13
@viceice viceice merged commit 8d4ffe3 into renovatebot:main Oct 16, 2023
5 checks passed
@CedricCabessa CedricCabessa deleted the docker-cmd-file branch October 16, 2023 09:41
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.

None yet

2 participants