Skip to content

ci: request slack-cli-team review on release candidate pull requests - #725

Merged
zimeg merged 2 commits into
mainfrom
zimeg-ci-release-reviewers
Oct 27, 2026
Merged

zimeg merged 2 commits into
mainfrom
zimeg-ci-release-reviewers

Conversation

@zimeg

@zimeg zimeg commented Oct 27, 2026

Copy link
Copy Markdown
Member

Changelog

No changes for release notes.

Summary

This pull request requests a review from slackapi/slack-cli-team on the release candidate pull request that the release workflow opens. Follows up on #593.

  • The release pull request only changes files in docs/, so CODEOWNERS routes it to slackapi/developer-education and the CLI team is never requested.
  • Adds --reviewer "slackapi/slack-cli-team" to gh pr create when a new release pull request is opened.
  • Updates to an existing release pull request on later pushes to main don't request again, so the team isn't pinged on every merge.

Testing

Notes

Requirements

🤖 Generated with Claude Code

@zimeg zimeg added semver:patch Use on pull requests to describe the release version increment build M-T: Changes to compilation and CI processes labels Oct 27, 2026
@zimeg zimeg self-assigned this Oct 27, 2026
@codecov

codecov Bot commented Oct 27, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.08%. Comparing base (d903910) to head (dcd929c).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #725      +/-   ##
==========================================
- Coverage   78.09%   78.08%   -0.02%     
==========================================
  Files         238      238              
  Lines       18095    18095              
==========================================
- Hits        14131    14129       -2     
- Misses       3964     3966       +2     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@zimeg
zimeg marked this pull request as ready for review October 27, 2026 17:18
@zimeg
zimeg requested a review from a team as a code owner October 27, 2026 17:18
@zimeg zimeg added this to the Next Release milestone Oct 27, 2026
@zimeg
zimeg enabled auto-merge (squash) October 27, 2026 17:20
@zimeg
zimeg disabled auto-merge October 27, 2026 18:37
@zimeg
zimeg merged commit 154975a into main Oct 27, 2026
13 checks passed
@zimeg
zimeg deleted the zimeg-ci-release-reviewers branch October 27, 2026 18:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

build M-T: Changes to compilation and CI processes semver:patch Use on pull requests to describe the release version increment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant