fix: skip version bump requirement for test/doc-only changes#28
Merged
TheRealAgentK merged 2 commits intomasterfrom Apr 16, 2026
Merged
fix: skip version bump requirement for test/doc-only changes#28TheRealAgentK merged 2 commits intomasterfrom
TheRealAgentK merged 2 commits intomasterfrom
Conversation
The version checker computed an only_tests_docs flag but never used it, causing test-only PRs to fail with 'version not incremented'. Now: 1. recommend_bump() returns 'patch' early when only_tests_docs is true 2. check_version_bump() skips the version check entirely when only tests/docs/requirements changed and the config is unchanged
TheRealAgentK
force-pushed
the
fix/version-check-test-only
branch
from
2a46d21 to
c8c5766
Compare
Shubhank-Jonnada
previously approved these changes
Apr 16, 2026
Shubhank-Jonnada
approved these changes
Apr 16, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Problem
The version checker fails PRs that only add/modify test files, docs, or requirements — demanding a version bump even though no integration code changed.
The
only_tests_docsflag was already computed inget_diff_stats()but never used anywhere.Fix
recommend_bump()— returns"patch"early whenonly_tests_docsis true, preventing test fileclass/deflines from being counted as "new features" (which was recommending a minor bump)check_version_bump()— skips the version check entirely when only tests/docs/requirements changed and the config (actions + auth) is unchanged. Prints✅ No version bump needed (only tests/docs changed)Context
Discovered while testing PR Autohive-AI/autohive-integrations#250 which adds unit tests to 5 integrations — CI was failing with "Version not incremented" for all 5 even though only
tests/files were added.