Skip to content

Configure classic publish notifications#2107

Merged
colincasey merged 1 commit into
mainfrom
classic-publish-notifications
Jun 16, 2026
Merged

Configure classic publish notifications#2107
colincasey merged 1 commit into
mainfrom
classic-publish-notifications

Conversation

@colincasey

Copy link
Copy Markdown
Contributor

Summary

Configures the classic Python buildpack into the publish notifications added to the shared _classic-buildpack-publish.yml workflow (heroku/languages-github-actions#370, already merged):

  • Passes the Linguist GitHub App (app_id + app_private_key) so the workflow's notify-start / notify-failure jobs can comment on the triggering "Prepare release" PR and author the GitHub Release as the App — matching the CNB release automation.
  • Passes the BUILDPACK_RELEASE_FAILURE_WEBHOOK_URL repo secret as the workflow's slack_webhook_url input, so production publish failures post to #heroku-languages-ops.

The reusable workflow now declares app_id (input) plus app_private_key / slack_webhook_url (secrets) as required, so a caller that doesn't pass them errors at the workflow_call. This PR supplies all three.

Prerequisites

  • vars.LINGUIST_GH_APP_ID and secrets.LINGUIST_GH_PRIVATE_KEY — org-level, already used by prepare_release.yml.
  • BUILDPACK_RELEASE_FAILURE_WEBHOOK_URL repo secret (Slack Workflow Builder webhook for #heroku-languages-ops) — must be set on this repo before the next release (will be provisioned manually).

W-22900983

@colincasey colincasey self-assigned this Jun 11, 2026
@colincasey colincasey marked this pull request as ready for review June 11, 2026 15:50
@colincasey colincasey requested a review from edmorley as a code owner June 11, 2026 15:50
@colincasey colincasey enabled auto-merge (squash) June 11, 2026 18:15
@colincasey colincasey merged commit 7db11ed into main Jun 16, 2026
8 of 13 checks passed
@colincasey colincasey deleted the classic-publish-notifications branch June 16, 2026 11:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants