Skip to content

Commit 200f1cb

Browse files
authored
feat: enable dry run support for office hours bot (hiero-ledger#1426) (hiero-ledger#1446)
Signed-off-by: cheese-cakee <farzanaman99@gmail.com>
1 parent 58a3a82 commit 200f1cb

2 files changed

Lines changed: 20 additions & 3 deletions

File tree

.github/workflows/bot-office-hours.yml

Lines changed: 14 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -3,6 +3,16 @@ name: PythonBot - Office Hour Reminder
33
on:
44
schedule:
55
- cron: "0 10 * * 3"
6+
workflow_dispatch:
7+
inputs:
8+
dry_run:
9+
description: "Run in dry-run mode (log only, no comments posted)"
10+
required: false
11+
default: "true"
12+
type: choice
13+
options:
14+
- "true"
15+
- "false"
616

717
permissions:
818
contents: read
@@ -11,6 +21,9 @@ permissions:
1121
jobs:
1222
office-hour-reminder:
1323
runs-on: ubuntu-latest
24+
concurrency:
25+
group: office-hour-reminder
26+
cancel-in-progress: false
1427
steps:
1528
- name: Harden the runner
1629
uses: step-security/harden-runner@20cf305ff2072d973412fa9b1e3a4f227bda3c76 #2.14.0
@@ -23,6 +36,6 @@ jobs:
2336
- name: Check Schedule and Notify
2437
env:
2538
GH_TOKEN: ${{ secrets.GITHUB_TOKEN }}
26-
DRY_RUN: "false" # will post
39+
DRY_RUN: ${{ github.event_name == 'workflow_dispatch' && inputs.dry_run || 'false' }}
2740
run: |
2841
bash .github/scripts/bot-office-hours.sh

CHANGELOG.md

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,11 @@ This changelog is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.
77
## [Unreleased]
88

99
### Added
10+
1011
- Auto-assignment bot for beginner-labeled issues with `/assign` command support and helpful reminders. (#1368)
1112
- Added comprehensive docstring to `FeeAssessmentMethod` enum explaining inclusive vs exclusive fee assessment methods with usage examples. (#1391)
1213
- Added comprehensive docstring to `TokenType` enum explaining fungible vs non-fungible tokens with practical use cases. (#1392)
14+
- Enable dry run support for office hours bot via `workflow_dispatch` trigger for testing without posting comments. (#1426)
1315

1416
- Added a notification workflow that alerts the support team when an issue is labeled as a Good First Issue Candidate.[(#1296)]
1517
- Added comprehensive training documentation for the `Query` class, covering execution flow, payments, retries, and building child queries. (#1238)
@@ -98,6 +100,7 @@ This changelog is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.
98100
- Added prompt for coderabbit to review `Query` and it's sub-classes.
99101

100102
### Changed
103+
101104
- Enable CodeRabbit walkthrough mode by default to improve PR review visibility (#1439)
102105
- Remove the commented out blocks in config.yml (#1435)
103106
- Renamed `.github/scripts/check_advanced_requirement.sh` to `bot-advanced-check.sh` for workflow consistency (#1341)
@@ -108,7 +111,7 @@ This changelog is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.
108111
- Renamed bot-inactivity workflow files to remove "-phase" suffix since the process no longer uses phased execution (#1339)
109112
- Renamed the GitHub notify team script to match its corresponding workflow filename for better maintainability (#1338)
110113
- style: apply black formatting to examples (#1299)
111-
-Update GitHub workflow names in `.github/workflows/bot-workflows.yml` to match correct references [(#1284)]
114+
-Update GitHub workflow names in `.github/workflows/bot-workflows.yml` to match correct references [(#1284)]
112115
- Renamed templates for improved clarity [(#1265)]
113116
- Updated Good First Issue notifications to trigger only after the first comment is posted, reducing noise on unassigned issues.(#1212)
114117
- Bumped requests from 2.32.3 to 2.32.4 to 2.32.5
@@ -165,9 +168,10 @@ This changelog is based on [Keep a Changelog](https://keepachangelog.com/en/1.1.
165168
- Refactored `examples/account/account_create_transaction_create_with_alias.py` and `examples/account/account_create_transaction_evm_alias.py` to use the native `AccountInfo.__str__` method for printing account details, replacing manual JSON serialization. ([#1263](https://github.com/hiero-ledger/hiero-sdk-python/issues/1263))
166169

167170
### Fixed
171+
168172
- Improved filename-related error handling with clearer and more descriptive error messages.(#1413)
169173
- Good First Issue bot no longer posts `/assign` reminders for repository collaborators. (#1367)
170-
- GFI workflow casing
174+
- GFI workflow casing
171175
- Update `bot-workflows.yml` to trigger only on open PRs with failed workflows; ignore closed PRs and branches without open PRs.
172176
- Fixed step-security/harden-runner action SHA in merge conflict bot workflow (#1278)
173177
- Fixed the README account balance example to use correct SDK APIs and provide a runnable testnet setup. (#1250)

0 commit comments

Comments
 (0)