chore: relax version-tracking tests per roadmap and CLAUDE.md policies#7
Merged
Merged
Conversation
Two test relaxations in this PR: 1. Remove test_current_matches_plugin_version Per standards/versioning.md (Developer-Tools-Directory v1.8.0), patch releases do not get themed roadmap table rows. The test was checking for behavior we explicitly do not want. 2. Remove test_claude_md_version Steam's CLAUDE.md uses hardcoded (vX.Y.Z) strings rather than the **Version:** pattern that release-doc-sync@v1.0 recognizes. Rather than retrofit CLAUDE.md to a different convention as a side effect of CI test cleanup, accept Steam's existing documentation style and stop enforcing version alignment in CLAUDE.md. If Steam adopts **Version:** convention later, the test can be reintroduced. The complementary tests that DO get auto-maintained by release-doc-sync remain: - test_changelog_has_current_version (passes) - test_roadmap_current_version (passes) Closes #5. Refs TMHSDigital/Developer-Tools-Directory#5. Signed-off-by: 154358121+TMHSDigital@users.noreply.github.com Made-with: Cursor
Contributor
Dependency Review✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.Scanned FilesNone |
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
1 participant
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.
chore: relax version-tracking tests per roadmap and CLAUDE.md policies
Two test relaxations in this PR:
Remove test_current_matches_plugin_version
Per standards/versioning.md (Developer-Tools-Directory v1.8.0),
patch releases do not get themed roadmap table rows. The test
was checking for behavior we explicitly do not want.
Remove test_claude_md_version
Steam's CLAUDE.md uses hardcoded (vX.Y.Z) strings rather than the
Version: pattern that release-doc-sync@v1.0 recognizes. Rather
than retrofit CLAUDE.md to a different convention as a side effect
of CI test cleanup, accept Steam's existing documentation style
and stop enforcing version alignment in CLAUDE.md.
If Steam adopts Version: convention later, the test can be
reintroduced.
The complementary tests that DO get auto-maintained by
release-doc-sync remain:
Closes #5.
Refs TMHSDigital/Developer-Tools-Directory#5.
Signed-off-by: 154358121+TMHSDigital@users.noreply.github.com