Skip to content

fix(resources): pr build status - find 'failure' state in status checks and log#14732

Merged
mcdurdin merged 5 commits intomasterfrom
fix/resources/14731-pr-build-status-ignoring-failure-state
Sep 11, 2025
Merged

fix(resources): pr build status - find 'failure' state in status checks and log#14732
mcdurdin merged 5 commits intomasterfrom
fix/resources/14731-pr-build-status-ignoring-failure-state

Conversation

@mcdurdin
Copy link
Copy Markdown
Member

Fixes: #14731
Build-bot: skip
Test-bot: skip

…ks and log

Fixes: #14731
Build-bot: skip
Test-bot: skip
@keymanapp-test-bot
Copy link
Copy Markdown

User Test Results

Test specification and instructions

User tests are not required

Copy link
Copy Markdown
Contributor

@ermshiperete ermshiperete left a comment

Choose a reason for hiding this comment

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

LGTM

local SCRIPT_CLONE=$(sed -n '/CLONE:START/,/CLONE:END/{//!p}' < "$MJS_FILE")
local SCRIPT_SUFFIX=$(sed -n '/CLONE-COMMENTED:START/,/CLONE-COMMENTED:END/{//!p}' < "$MJS_FILE" | sed 's/\/\/ //g')
echo -e "# GENERATED FILE - DO NOT EDIT!\n$YML_PREFIX\n$SCRIPT_CLONE\n$SCRIPT_SUFFIX\n" > "$YML_FILE"
echo $'# GENERATED FILE - DO NOT EDIT!\n'"$YML_PREFIX"$'\n'"$SCRIPT_CLONE"$'\n'"$SCRIPT_SUFFIX"$'\n' > "$YML_FILE"
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

I'm always learning something new 😄

@mcdurdin mcdurdin merged commit 52caaa4 into master Sep 11, 2025
6 checks passed
@mcdurdin mcdurdin deleted the fix/resources/14731-pr-build-status-ignoring-failure-state branch September 11, 2025 12:57
@github-project-automation github-project-automation bot moved this from Todo to Done in Keyman Sep 11, 2025
@mcdurdin
Copy link
Copy Markdown
Member Author

Ah, I updated the .yml but not the .mjs, will revisit this shortly

mcdurdin added a commit that referenced this pull request Sep 11, 2025
@keyman-server
Copy link
Copy Markdown
Collaborator

Changes in this pull request will be available for download in Keyman version 19.0.117-alpha

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

bug(resources): pr build status accepted failing builds

3 participants