Skip to content

Commit 630267a

Browse files
authored
fix: handle string boolean in broken links workflow (#1242)
Signed-off-by: Danish Ali <[email protected]> Signed-off-by: Danish Ali Shaikh <[email protected]>
1 parent c7f5212 commit 630267a

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

.github/workflows/cron-check-broken-links.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -44,7 +44,7 @@ jobs:
4444
script: |
4545
// Determine if this is a dry run
4646
const isManual = context.eventName === 'workflow_dispatch';
47-
const dryRun = isManual ? context.payload.inputs.dry_run === true : false;
47+
const dryRun = isManual ? String(context.payload.inputs.dry_run).toLowerCase() === 'true' : false;
4848
4949
// Labels configuration
5050
const targetLabels = ['broken-markdown-links', 'automated'];

CHANGELOG.md

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -117,6 +117,7 @@ This changelog is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.
117117
- Fix unit test tet_query.py
118118
- TLS Hostname Mismatch & Certificate Verification Failure for Nodes
119119
- Workflow does not contain permissions for `pr-check-test-files` and `pr-check-codecov`
120+
- Fixed `cron-check-broken-links.yml` string parsing issue in context input `dry_run` (#1235)
120121
- Flaky tests by disabling TLS in mock Hedera nodes in `mock_server.py`
121122

122123

0 commit comments

Comments
 (0)