Skip to content

Comments

ci: add Slack notification for merged PRs#532

Merged
factory-ben merged 3 commits intomainfrom
feature/pr-merge-slack-notification
Dec 17, 2025
Merged

ci: add Slack notification for merged PRs#532
factory-ben merged 3 commits intomainfrom
feature/pr-merge-slack-notification

Conversation

@factory-ben
Copy link
Collaborator

Adds a GitHub Actions workflow that posts to Slack when PRs are merged to main.

Details posted:

  • PR title (linked)
  • Author and merge commit SHA

Uses the existing SLACK_WEBHOOK_URL secret.

factory-ben and others added 2 commits December 17, 2025 21:14
- Rename 'Web Platform' section to 'Factory App'
- Remove Factory Bridge documentation (replaced by desktop app)
- Update overview and quickstart for new app experience
- Add download links for Mac and Windows
- Add redirects for old Bridge URLs

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
@factory-droid
Copy link
Contributor

factory-droid bot commented Dec 17, 2025

Droid finished @factory-ben's task —— View job


Code Review Complete

Reviewed the PR changes including the new Slack notification workflow and documentation updates.

Finding:

  • Posted 1 inline comment on .github/workflows/pr-merge-slack.yml regarding missing error handling on the curl command (the notification could silently fail if Slack returns an error).

Co-authored-by: factory-droid[bot] <138933559+factory-droid[bot]@users.noreply.github.com>
@factory-sam factory-sam self-requested a review December 17, 2025 21:38
@factory-ben factory-ben requested review from factory-sam and removed request for factory-sam December 17, 2025 21:40
@factory-ben factory-ben merged commit 074ee96 into main Dec 17, 2025
2 checks passed
@factory-ben factory-ben deleted the feature/pr-merge-slack-notification branch December 17, 2025 21:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants