Repository navigation
Remove unused gardener workflows and Slack notifications - #9022
Merged
Merged
Conversation
The workflow has never run, its trigger label was never applied in this repo, and it pins the claude-code-action release that no longer supports the configured model. Co-authored-by: Claude Code <claude-code@anthropic.com>
4 tasks
isaacroldan
marked this pull request as ready for review
October 30, 2026 10:25
This was referenced Oct 30, 2026
Deletes gardener-notify-event.yml and gardener-notify-slack.yml, which posted devtools-gardener-labelled items and every Dependabot PR to #devtools-gardener-backlog and auto-labelled Dependabot PRs. Co-authored-by: Claude Code <claude-code@anthropic.com>
byrichardpowell
approved these changes
Oct 30, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
WHY are these changes introduced?
This PR is part of the gardener cleanup and removes two things.
Gardener - Investigate Issue(.github/workflows/gardener-investigate-issue.yml) has never done anything in this repo:disabled_manuallysince 2026-10-26.devtools-investigate-for-gardener, has been applied 0 times in Shopify/cli.anthropics/claude-code-action@36a69b6(v1.0.134, which bundles Claude Code 2.1.161). That is the same pin that now breaksmaintenance-prs: runs 40249213977 and 40426069655 fail withClaude Code 2.1.161 does not support this model; version 2.1.280 or newer is required. If anyone re-enabled this workflow, it would fail the same way.The evidence comes from a read-only inventory of Dev Experience automations collected on 2026-10-27 (workflow state, run history and label usage via
gh).The gardener Slack notifications are being retired. On 2026-10-30 Isaac decided to remove them. In this repo they posted only
devtools-gardener-labelled items and every Dependabot PR to #devtools-gardener-backlog. Each post was a bare linked title, and together they made up most of the channel's noise. From now on, the Torrent gardener-triage routine (GARDTRI) will surface new issues and Dependabot PRs.WHAT is this pull request doing?
Remove
gardener-investigate-issue.yml. No docs or agent files reference this workflow. Theinvestigating-github-issuesskill stays, because it is general-purpose and not specific to this workflow.Remove the gardener Slack notifications. This deletes both notification workflows:
.github/workflows/gardener-notify-event.yml: stashed the event payload whendevtools-gardenerwas applied or when Dependabot opened a PR..github/workflows/gardener-notify-slack.yml: ran onworkflow_run, addeddevtools-gardenerto Dependabot PRs and posted a linked title to Slack.After merge, Dependabot PRs will no longer get the
devtools-gardenerlabel automatically. Nothing else in the repo depends only on these two workflows. I grepped forgardener-notify,GARDENER_SLACK,SLACK_GARDENERanddevtools-gardenerand checked helper scripts, label/config files and docs. This PR doesn't touch thedevtools-gardenerlabel, the other workflows or the Torrent gardener-triage routine (GARDTRI).Secrets and variables the removed workflows used (none are deleted here; removing them is left to the maintainers):
secrets.SLACK_GARDENER_BOT_TOKEN: used bygardener-investigate-issue.ymlandgardener-notify-slack.yml, which are both removed here.maintenance-prs.ymlonmainstill uses it (Remove maintenance-prs workflow and automated task prompts #9047 removes that workflow).vars.GARDENER_SLACK_CHANNEL_ID: used by the same two workflows. Once this PR merges, nothing else in the repo uses it.This is separate from #9008 (merged), which removed
gardener-backlog-prs.yml. This PR doesn't touch that file. It only changes workflows, so there's no changeset.How to manually test your changes?
There is nothing to test locally because this only deletes workflows. After merge:
Gardener - Investigate Issue,Gardener - Notify EventandGardener - Notify Slackshould no longer show up as workflows defined onmain.devtools-gardener, or Dependabot opening a PR, should no longer post to Slack.Checklist
patchfor bug fixes ·minorfor new features ·majorfor breaking changes) and added a changeset withpnpm changeset add