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

Fix issue with parsing docker images with a tag in the <prefix>_<year><month><day>.<version> format #8357

Closed
wants to merge 3 commits into from

Conversation

jpinz
Copy link
Contributor

@jpinz jpinz commented Nov 7, 2023

Fix issue with parsing docker images with a tag in the _. format.
Included specs to test with.
We were facing issues with the version-update:semver-major ignore condition, and this appears to fix it.

@jpinz jpinz requested a review from a team as a code owner November 7, 2023 16:55
@github-actions github-actions bot added the L: docker Docker containers label Nov 7, 2023
@jpinz
Copy link
Contributor Author

jpinz commented Nov 7, 2023

Appears that this is blocked until #8192 is merged

@brettfo
Copy link
Contributor

brettfo commented Nov 15, 2023

The contents of this PR are up-to-date as of internal commit 15e3b8d09374468e40d8d77bafc1c01178db444c.

@jpinz jpinz self-assigned this Nov 28, 2023
@JamieMagee
Copy link
Contributor

@jpinz Can you recreate this from a branch instead of a fork please?

@jpinz jpinz closed this Nov 30, 2023
@jpinz
Copy link
Contributor Author

jpinz commented Nov 30, 2023

@jpinz Can you recreate this from a branch instead of a fork please?

Done! #8500

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
L: docker Docker containers
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants