Skip to content

Conversation

@pando85
Copy link

@pando85 pando85 commented Jan 17, 2026

Resolves #27862.

Changes

Added support for dated container image version formats in Debian versioning, similar to existing Ubuntu support. This includes parsing, validation, stability checks, and comparison logic for versions like bookworm-20230816 and bullseye-20220101.2.

Key changes:

  • Updated isDatedCodeName to validate codenames against Debian distro info
  • Enhanced DebianVersioningApi with dated version handling in isValid, isStable, _parse, equals, and isGreaterThan
  • Added comprehensive unit tests for dated version functionality

Context

Please select one of the following:

AI assistance disclosure

Did you use AI tools to create any part of this pull request?

Please select one option and, if yes, briefly describe how AI was used (e.g., code, tests, docs) and which tool(s) you used.

  • No — I did not use AI for this contribution.
  • Yes — minimal assistance (e.g., IDE autocomplete, small code completions, grammar fixes).
  • Yes — substantive assistance (AI-generated non‑trivial portions of code, tests, or documentation).
  • [] Yes — other (please describe):

Documentation (please check one with an [x])

  • I have updated the documentation, or
  • [] No documentation update is required

How I've tested my work (please select one)

I have verified these changes via:

  • Code inspection only, or
  • Newly added/modified unit tests, or
  • No unit tests, but ran on a real repository, or
  • Both unit tests + ran on a real repository

The public repository: N/A (local development)

@pando85 pando85 force-pushed the feat/support-debian-versioning branch 6 times, most recently from 107cb22 to c0df406 Compare January 19, 2026 16:25
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.

missing coverage

@pando85 pando85 force-pushed the feat/support-debian-versioning branch 3 times, most recently from a3a1327 to c3845a4 Compare January 25, 2026 07:11
@pando85
Copy link
Author

pando85 commented Jan 25, 2026

I added a small test to cover the two lines that were missing. It should be ready to merge now.

Resolves renovatebot#27862.

Signed-off-by: Alexander Gil <pando855@gmail.com>
@pando85 pando85 force-pushed the feat/support-debian-versioning branch 5 times, most recently from 5b0e3b5 to 03831d6 Compare January 27, 2026 09:15
Signed-off-by: Alexander Gil <pando855@gmail.com>
@pando85 pando85 closed this Jan 27, 2026
@pando85 pando85 force-pushed the feat/support-debian-versioning branch from 03831d6 to 7f649d7 Compare January 27, 2026 10:00
@pando85 pando85 reopened this Jan 27, 2026
@pando85
Copy link
Author

pando85 commented Jan 27, 2026

Finally, all green. Ping @viceice

${'buster-20220101'} | ${'buster'}
${'bullseye-20220101'} | ${'bullseye'}
${'bookworm-20230816'} | ${'bookworm'}
${'bookworm-20230816.1'} | ${'bookworm'}
Copy link
Collaborator

Choose a reason for hiding this comment

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

here we return the codename but in the previous isDatedCodename we return false for buster-20220101.123..why this difference ?

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.

Implement support of "<codename>-<date>" version format for Debian docker images

3 participants