Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support image digests in FROM instructions #31

Merged
merged 3 commits into from
Oct 31, 2024

Conversation

thundergolfer
Copy link
Contributor

This is a re-up of #15, but updates the test code to be compatible with current APIs and also includes some additional validation of the digest part (with tests).

@thundergolfer
Copy link
Contributor Author

Sorry my IDE introduced a some indenting changes here, which artificially blows up the diff. I'll make sure to remove those before merge.

@zubieta
Copy link

zubieta commented Aug 11, 2024

It'll be pretty useful to have this PR or #15 merged to unblock dprint/dprint-plugin-dockerfile#9.

@jayvdb
Copy link

jayvdb commented Sep 12, 2024

ping @jonmcquillan

@jonmcquillan
Copy link
Contributor

jonmcquillan commented Oct 30, 2024

Sorry it took so long to get to looking at this. Can you merge in master to kick off the github actions I migrated the repo to? @jayvdb

@jayvdb
Copy link

jayvdb commented Oct 30, 2024

ping @thundergolfer ;-)

@thundergolfer
Copy link
Contributor Author

@jayvdb done

@jonmcquillan jonmcquillan merged commit bc52c98 into HewlettPackard:master Oct 31, 2024
1 check passed
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants