Skip to content

fix: Ensure validation failures are reported to the release dashboard.#191

Merged
iphydf merged 1 commit into
TokTok:masterfrom
iphydf:validate
Feb 14, 2026
Merged

fix: Ensure validation failures are reported to the release dashboard.#191
iphydf merged 1 commit into
TokTok:masterfrom
iphydf:validate

Conversation

@iphydf
Copy link
Copy Markdown
Member

@iphydf iphydf commented Feb 13, 2026

When validate_pr.py fails, it now raises an InvalidState exception instead of exiting, allowing create_release.py to catch the failure and report it to the release tracking issue.


This change is Reviewable

@iphydf iphydf added this to the v0.8.4 milestone Feb 13, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions Bot commented Feb 13, 2026

Tip

Preview URL:

@github-actions github-actions Bot added the bug Bug fix for the user, not a fix to a build script label Feb 13, 2026
When `validate_pr.py` fails, it now raises an `InvalidState` exception
instead of exiting, allowing `create_release.py` to catch the failure
and report it to the release tracking issue.
@iphydf iphydf marked this pull request as ready for review February 13, 2026 21:35
Copy link
Copy Markdown
Member Author

@iphydf iphydf left a comment

Choose a reason for hiding this comment

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

@iphydf reviewed 4 files and all commit messages.
Reviewable status: :shipit: complete! all files reviewed, all discussions resolved (waiting on @iphydf).

@iphydf iphydf merged commit bd8b5ba into TokTok:master Feb 14, 2026
40 of 41 checks passed
@iphydf iphydf deleted the validate branch February 14, 2026 13:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Bug fix for the user, not a fix to a build script

Development

Successfully merging this pull request may close these issues.

2 participants