test: Add single-parser test optimization to CI#1148
Open
nathan-stender wants to merge 3 commits intomainfrom
Open
test: Add single-parser test optimization to CI#1148nathan-stender wants to merge 3 commits intomainfrom
nathan-stender wants to merge 3 commits intomainfrom
Conversation
Detect when changes are isolated to a single parser directory and run only that parser's tests, reducing CI time by 10-30x for ~30% of PRs. Falls back to running all tests when changes affect multiple parsers or shared code. Co-Authored-By: Claude Opus 4.1 <noreply@anthropic.com>
- Include test files in parser detection (tests/parsers/**) - Add SUPPORTED_INSTRUMENT_SOFTWARE.adoc to non-parser filter - Update detection logic to handle both src and test parser files - Exclude utils from parser detection in src directory This ensures the workflow correctly detects single-parser changes even when test data files are modified alongside parser code. Co-Authored-By: Claude Opus 4.1 <noreply@anthropic.com>
nathan-stender
changed the title
nathan-stender
changed the title
- Remove SUPPORTED_INSTRUMENT_SOFTWARE.adoc from non-parser filter (it's an output of parser changes, not an input that affects parsers) - Add test_table_contents validation to single-parser test runs to ensure the instrument table is updated when needed This correctly treats the instrument table as a consequence of parser changes rather than a trigger for running all tests. Single-parser test runs now also validate that the table is up-to-date. Co-Authored-By: Claude Opus 4.1 <noreply@anthropic.com>
nathan-stender
changed the title
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.
Summary
Changes
detect-changesjob usingdorny/paths-filterto analyze file changesTest plan
Impact
Based on analysis:
🤖 Generated with Claude Code