Thanks to visit codestin.com
Credit goes to github.com

Skip to content

feat: allow to override docker command and user#783

Merged
viceice merged 11 commits into
renovatebot:mainfrom
CedricCabessa:docker-cmd-file
Oct 16, 2023
Merged

feat: allow to override docker command and user#783
viceice merged 11 commits into
renovatebot:mainfrom
CedricCabessa:docker-cmd-file

Conversation

@CedricCabessa
Copy link
Copy Markdown
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
Copy Markdown
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
Copy Markdown
Member

viceice commented Jul 21, 2023

please add an e2e test

@CedricCabessa
Copy link
Copy Markdown
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 CedricCabessa requested a review from viceice July 24, 2023 07:14
@CedricCabessa
Copy link
Copy Markdown
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

Comment thread example/entrypoint.sh Outdated
Comment thread src/renovate.ts Outdated
@CedricCabessa
Copy link
Copy Markdown
Contributor Author

@viceice sorry for the delay, I was off 🌴

@CedricCabessa CedricCabessa requested a review from viceice August 7, 2023 09:02
@CedricCabessa
Copy link
Copy Markdown
Contributor Author

@viceice 👋 ;-)

Comment thread .github/workflows/build.yml
Comment thread README.md Outdated
Comment on lines +19 to +20
- [`docker-cmd-file`](#docker-cmd-file)
- [`docker-user`](#docker-user)
Copy link
Copy Markdown
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
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

done

Comment thread README.md Outdated
@CedricCabessa
Copy link
Copy Markdown
Contributor Author

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

@viceice
Copy link
Copy Markdown
Member

viceice commented Sep 14, 2023

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

use the re-request review button

Comment thread src/renovate.ts Outdated
Comment thread src/renovate.ts Outdated
Comment thread src/renovate.ts Outdated
@CedricCabessa CedricCabessa requested a review from viceice October 16, 2023 09:03
@viceice viceice enabled auto-merge (squash) October 16, 2023 09:13
@viceice viceice merged commit 8d4ffe3 into renovatebot:main Oct 16, 2023
@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.

2 participants