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

ci(fix): Properly auto-update Dockerfiles using a build arg #309

Merged
merged 2 commits into from
Aug 29, 2024

Conversation

alessfg
Copy link
Collaborator

@alessfg alessfg commented Aug 29, 2024

Proposed changes

Dependabot will not generate PRs for Dockerfiles that include a build argument within their FROM instruction. This PR fixes that issue.

Checklist

Before creating a pull request (PR), run through this checklist and mark each as complete:

Dependabot will not generate PRs for Dockerfiles that include a build argument within their FROM instruction
@alessfg alessfg self-assigned this Aug 29, 2024
@alessfg alessfg requested a review from 4141done August 29, 2024 21:29
@alessfg alessfg added the bug Something isn't working label Aug 29, 2024
Dockerfile.plus Outdated
@@ -1,8 +1,8 @@
ARG RELEASE=bookworm
FROM debian:${RELEASE}-slim@sha256:f528891ab1aa484bf7233dbcc84f3c806c3e427571d75510a9d74bb5ec535b33
# ARG RELEASE=bookworm # Removing for the time being since it doesn't play well with Dependabot
Copy link
Collaborator

Choose a reason for hiding this comment

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

If we are removing it let's just delete it for now

Copy link
Collaborator

@4141done 4141done left a comment

Choose a reason for hiding this comment

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

👍 🐳 👍
Let's delete rather than commenting out - we can always re-add it based on our git history

@alessfg
Copy link
Collaborator Author

alessfg commented Aug 29, 2024

Done!

@alessfg alessfg merged commit 5d60dd0 into main Aug 29, 2024
10 checks passed
@alessfg alessfg deleted the fix-dockerfile-update branch August 29, 2024 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants