Skip to content

add pre-commit check for required version bumps - #770

Open
msarahan wants to merge 3 commits into
rapidsai:mainfrom
msarahan:codex/check-feature-version-bumps
Open

msarahan wants to merge 3 commits into
rapidsai:mainfrom
msarahan:codex/check-feature-version-bumps

Conversation

@msarahan

@msarahan msarahan commented Sep 9, 2026

Copy link
Copy Markdown
Contributor

It is common for people to change code in a feature and forget to bump the version. When that happens, the new images are not pushed to GHCR. This check is intended to assist with that.

It accounts for the following possible situations:

  • forgotten version bump
  • version value lower than high watermark for XX.YY release series
  • no-op if manual version bump has been done

This has one config setting - whether to automatically fix files, like a linter.

      # Opt in to automatic feature version updates in this checkout with:
      # git config devcontainers.auto-bump-feature-versions true

Missing feature version updates are mechanical but easy to overlook, especially after a release reset makes the visible patch number lower than a version already published from the same series. Allow contributors to opt into formatter-style fixes that select the next patch above that same-series high-water mark, update the affected manifests, and stop the commit so the generated changes can be reviewed and staged before retrying. Keep pull-request validation read-only and refuse to overwrite unstaged manifest edits.
Auto-bump must remain a fallback for missing or invalid version changes, not increment a version that a contributor already selected correctly. Exercise the checker with auto-bump enabled and a valid staged version update, and assert that it succeeds without modifying that version again.
@msarahan
msarahan requested a review from a team as a code owner September 9, 2026 19:07
@msarahan
msarahan requested review from KyleFromNVIDIA and removed request for a team September 9, 2026 19:07
"name": "NVIDIA RAPIDS devcontainer build utilities",
"id": "rapids-build-utils",
"version": "26.12.0",
"version": "26.12.1",

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Does this version need to be bumped in this PR?

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I don't understand why it was added. I'll look into why this happened.

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

This is a catch-up bump from the forward-merge in #768 . It should have been done there. Would you like me to split it off into its own PR, since it is noise in this PR?

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.

2 participants