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 using non-slim renovate image #626

Merged
merged 7 commits into from
Jan 23, 2023

Conversation

avorima
Copy link
Contributor

@avorima avorima commented Jul 21, 2022

This probably is my first time writing TypeScript, so please bear with me.

Added optional useSlim action input. If it is set to false, then the normal renovate/renovate image will be used. The default behavior remains unchanged.

Fixes: #625

@avorima avorima marked this pull request as ready for review July 21, 2022 09:28
src/input.ts Outdated Show resolved Hide resolved
src/input.ts Outdated Show resolved Hide resolved
src/input.ts Outdated Show resolved Hide resolved
src/input.ts Outdated Show resolved Hide resolved
src/docker.ts Outdated Show resolved Hide resolved
@avorima avorima force-pushed the configurable-slim-image branch 2 times, most recently from 78408f9 to 00169c6 Compare January 4, 2023 11:13
Signed-off-by: Mario Valderrama <mario.valderrama@ionos.com>
@avorima
Copy link
Contributor Author

avorima commented Jan 4, 2023

@viceice Thanks for your review. I rebased and amended my commit to get the latest changes from main. PTAL

@viceice
Copy link
Member

viceice commented Jan 4, 2023

@viceice Thanks for your review. I rebased and amended my commit to get the latest changes from main. PTAL

Please use update with merge commit next time, so we don't loose review history. We squash the PR anyways, so don't worry about PR history

@viceice
Copy link
Member

viceice commented Jan 4, 2023

Also use the re-request review feature when a new review is needed
image

@avorima avorima requested a review from viceice January 4, 2023 16:36
src/docker.ts Outdated Show resolved Hide resolved
src/docker.ts Show resolved Hide resolved
@avorima avorima requested a review from viceice January 4, 2023 20:45
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.

Remember that this can fail at runtime when the full image publish is delayed due to issues.

@viceice viceice changed the title Allow using non-slim renovate image feat: Allow using non-slim renovate image Jan 23, 2023
@viceice
Copy link
Member

viceice commented Jan 23, 2023

Please resolve fixed discussions

src/get-version.ts Outdated Show resolved Hide resolved
@viceice viceice enabled auto-merge (squash) January 23, 2023 14:36
@viceice viceice merged commit e18dc08 into renovatebot:main Jan 23, 2023
@avorima avorima deleted the configurable-slim-image branch January 23, 2023 15:12
@viceice viceice added the regression Issue about a regression bug, or the PR caused it label Feb 3, 2023
@kayman-mk kayman-mk mentioned this pull request Feb 15, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
regression Issue about a regression bug, or the PR caused it
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Allow using non-slim renovate image
2 participants