Review and fix GitHub Actions workflows#72
Merged
MarkusNeusinger merged 3 commits intomainfrom Dec 1, 2025
Merged
Conversation
Owner
MarkusNeusinger
commented
Dec 1, 2025
MarkusNeusinger
commented
- Fix regex patterns to match actual format: Parent Issue: and Sub-Issue:
- Add sub_issue_number and library fields to plot_metadata.json in gen-preview.yml
- Update bot-sync-status.yml to search for correct "Parent Issue" format
- Update bot-auto-merge.yml, gen-update-plot.yml, bot-ai-review.yml patterns
- Fix regex patterns to match actual format: **Parent Issue:** and **Sub-Issue:** - Add sub_issue_number and library fields to plot_metadata.json in gen-preview.yml - Update bot-sync-status.yml to search for correct "Parent Issue" format - Update bot-auto-merge.yml, gen-update-plot.yml, bot-ai-review.yml patterns
Codecov Report✅ All modified and coverable lines are covered by tests. 📢 Thoughts on this report? Let us know! |
Contributor
There was a problem hiding this comment.
Pull request overview
This PR fixes inconsistent regex patterns in GitHub Actions workflows to correctly parse issue references from PR bodies. The actual format uses markdown bold markers (**Parent Issue:** #N and **Sub-Issue:** #N), but many workflows were searching for these patterns without the asterisks.
Key changes:
- Updated
grep -oPregex patterns to escape asterisks (\*\*) for extracting parent/sub-issue numbers from PR bodies - Added
sub_issue_numberandlibraryfields toplot_metadata.jsonin gen-preview.yml for downstream consumption - Updated inline comments to reflect correct format strings
Reviewed changes
Copilot reviewed 5 out of 5 changed files in this pull request and generated 2 comments.
Show a summary per file
| File | Description |
|---|---|
.github/workflows/gen-update-plot.yml |
Fixed regex patterns for Sub-Issue and Parent Issue extraction to match markdown bold format |
.github/workflows/gen-preview.yml |
Fixed regex patterns and added sub_issue_number/library fields to metadata JSON for AI review workflow |
.github/workflows/bot-sync-status.yml |
Fixed regex pattern for Parent Issue extraction and updated comments (but search query on line 108 still needs fixing) |
.github/workflows/bot-auto-merge.yml |
Fixed regex patterns for Sub-Issue and Parent Issue extraction (but search query on line 234 still needs fixing) |
.github/workflows/bot-ai-review.yml |
Fixed regex pattern for Sub-Issue extraction and updated comment |
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
MarkusNeusinger
deleted the
claude/fix-github-actions-014fvFJLEVhTP3Nj66BCNymb
branch
Contributor
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 5 out of 5 changed files in this pull request and generated no new comments.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
MarkusNeusinger
added a commit
that referenced
this pull request
Dec 1, 2025
- Fix regex patterns to match actual format: **Parent Issue:** and **Sub-Issue:** - Add sub_issue_number and library fields to plot_metadata.json in gen-preview.yml - Update bot-sync-status.yml to search for correct "Parent Issue" format - Update bot-auto-merge.yml, gen-update-plot.yml, bot-ai-review.yml patterns
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
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.