Repository navigation
Attach the Vercel build log and PR link to every Vercel Slack post - #2089
Open
marcleblanc2 wants to merge 4 commits into
Open
marcleblanc2 wants to merge 4 commits into
marcleblanc2 wants to merge 4 commits into
Conversation
The Vercel Slack app's deployment posts only link to Vercel, where the PR is behind a Vercel login. The workflow already attached the build log to the "failed to deploy" post with a link to the PR; now it does the same for successful builds, and for production deployments, linking the merged PR that introduced the commit. The PR link rides on the file upload's comment, so the bot needs no new scope. The script is now one run instead of three commands handing each other state through GITHUB_OUTPUT and a log file: PR lookup, Vercel, Slack, PR comment, in order. A Slack error is a warning annotation, so the PR still hears about a failed build, as continue-on-error did before. Amp-Thread-ID: https://ampcode.com/threads/T-01a130c9-ad60-74db-aa70-8ebc50f43069 Co-authored-by: Amp <amp@ampcode.com>
Contributor
…ript Amp-Thread-ID: https://ampcode.com/threads/T-01a130c9-ad60-74db-aa70-8ebc50f43069 Co-authored-by: Amp <amp@ampcode.com>
…ck app Amp-Thread-ID: https://ampcode.com/threads/T-01a130c9-ad60-74db-aa70-8ebc50f43069 Co-authored-by: Amp <amp@ampcode.com>
Amp-Thread-ID: https://ampcode.com/threads/T-01a130c9-ad60-74db-aa70-8ebc50f43069 Co-authored-by: Amp <amp@ampcode.com>
marcleblanc2
marked this pull request as ready for review
October 12, 2026 16:58
marcleblanc2
enabled auto-merge (squash)
October 12, 2026 16:58
|
The latest updates on your projects. Learn more about Vercel for GitHub. 1 Skipped Deployment
|
This branch has not been deployed
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
1 participant
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.
Vercel's Slack app's deployment posts hide the PR behind a Vercel login, requiring too many steps to get from Slack to PR
The
Vercel build logworkflow had replied to Vercel's Slack app's posts with a link to the PR, and the build log, for only failed buildsThis PR changes the
Vercel build logworkflow to reply to Vercel's Slack app's posts with the same link to the PR, and the build log, for all PR builds, success or failureThis PR also renames files to align with the Slack bot's name