Skip to content

Commit b325a01

Browse files
nohwndCopilot
andauthored
Fix workflow config gaps: enable @nohwnd mentions, fix PR description instruction (#15770)
- daily-qa: Enable mentions and allow @nohwnd reference so digest notifications work - pr-iteration: Change PR description instruction from 'rewrite it' to 'flag in comment' (workflow lacks write permission to update PR body) Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
1 parent c7d4417 commit b325a01

3 files changed

Lines changed: 21 additions & 21 deletions

File tree

.github/workflows/daily-qa.lock.yml

Lines changed: 17 additions & 17 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

.github/workflows/daily-qa.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -23,8 +23,8 @@ imports:
2323
safe-outputs:
2424
noop:
2525
report-as-issue: false
26-
mentions: false
27-
allowed-github-references: []
26+
mentions: true
27+
allowed-github-references: ["@nohwnd"]
2828
create-issue:
2929
max: 1
3030
title-prefix: "Daily Maintenance Digest"

.github/workflows/pr-iteration.md

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -90,7 +90,7 @@ If triggered by `schedule` or `workflow_dispatch`, check ALL your PRs and iterat
9090
d. Make the requested changes
9191
e. Build and run tests to verify
9292
f. Push the fix
93-
g. Verify the PR description accurately describes the code changes. If it's a template placeholder or doesn't match the diff, rewrite it.
93+
g. Verify the PR description accurately describes the code changes. If it's a template placeholder or doesn't match the diff, flag it in a review comment with a suggested replacement description.
9494
h. Reply briefly to the review comment confirming what you changed
9595
4. If the comment is just a question or discussion, reply if you can add useful context. Otherwise noop.
9696

@@ -104,7 +104,7 @@ If triggered by `schedule` or `workflow_dispatch`, check ALL your PRs and iterat
104104
- Determine if it's a code issue (fix it) or infrastructure flake (comment and skip)
105105
- Push a fix if possible
106106
c. Check for unaddressed review comments. If any, address them.
107-
d. Check that the PR description accurately describes the code changes. If it's a template placeholder or doesn't match the diff, rewrite it.
107+
d. Check that the PR description accurately describes the code changes. If it's a template placeholder or doesn't match the diff, flag it in a comment with a suggested replacement description.
108108
e. Check for merge conflicts. If conflicted, merge main and push.
109109
3. Update cache-memory with the cleaned-up list (remove merged/closed PRs).
110110

0 commit comments

Comments
 (0)