Skip to content

Conversation

@sileht
Copy link
Member

@sileht sileht commented Oct 16, 2025

This adds some information on the job page on the matching scopes.

This is very useful for reporting what the action does and for debugging.

Copilot AI review requested due to automatic review settings October 16, 2025 10:24
@mergify mergify bot had a problem deploying to Mergify Merge Protections October 16, 2025 10:24 Failure
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This PR adds a GitHub Actions job summary feature that displays scope matching results in a formatted markdown table. The summary includes all scopes and indicates which ones matched the changes using checkmark and cross emojis.

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
mergify_cli/ci/scopes/cli.py Implements maybe_write_github_step_summary() function and integrates it into the detect() workflow
mergify_cli/tests/ci/scopes/test_cli.py Adds test coverage for the new GitHub step summary functionality

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@sileht
Copy link
Member Author

sileht commented Oct 16, 2025

CleanShot 2025-10-16 at 12 24 58

@mergify
Copy link
Contributor

mergify bot commented Oct 16, 2025

Merge Protections

Your pull request matches the following merge protections and will not be merged until they are valid.

🟢 🤖 Continuous Integration

Wonderful, this rule succeeded.
  • all of:
    • check-success=test (macos-15)
    • check-success=test (ubuntu-24.04)
    • check-success=test (windows-2025)

🟢 👀 Review Requirements

Wonderful, this rule succeeded.
  • any of:
    • #approved-reviews-by>=2
    • author = dependabot[bot]
    • author = renovate[bot]

🟢 Enforce conventional commit

Wonderful, this rule succeeded.

Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/

  • title ~= ^(fix|feat|docs|style|refactor|perf|test|build|ci|chore|revert)(?:\(.+\))?:

🟢 🔎 Reviews

Wonderful, this rule succeeded.
  • #changes-requested-reviews-by = 0
  • #review-requested = 0
  • #review-threads-unresolved = 0

@mergify
Copy link
Contributor

mergify bot commented Oct 16, 2025

🧪 CI Insights

Here's what we observed from your CI run for 01450ca.

🟢 All jobs passed!

But CI Insights is watching 👀

@mergify mergify bot requested a review from a team October 16, 2025 10:26
remyduthu
remyduthu previously approved these changes Oct 16, 2025
@sileht sileht force-pushed the devs/sileht/scopes-summary/I60d468996a12845f9e5c770f529fd33515242124 branch from d4c0315 to cd3e7af Compare October 16, 2025 10:38
@mergify mergify bot dismissed remyduthu’s stale review October 16, 2025 10:39

Pull request has been modified.

@mergify mergify bot had a problem deploying to Mergify Merge Protections October 16, 2025 10:39 Failure
jd
jd previously approved these changes Oct 16, 2025
@sileht sileht force-pushed the devs/sileht/scopes-summary/I60d468996a12845f9e5c770f529fd33515242124 branch from cd3e7af to 7eebbbc Compare October 16, 2025 12:41
@mergify mergify bot dismissed jd’s stale review October 16, 2025 12:42

Pull request has been modified.

@mergify mergify bot had a problem deploying to Mergify Merge Protections October 16, 2025 12:42 Failure
@mergify mergify bot requested a review from a team October 16, 2025 12:43
Change-Id: I60d468996a12845f9e5c770f529fd33515242124
@sileht sileht force-pushed the devs/sileht/scopes-summary/I60d468996a12845f9e5c770f529fd33515242124 branch from 7eebbbc to 01450ca Compare October 16, 2025 12:51
@mergify mergify bot deployed to Mergify Merge Protections October 16, 2025 12:52 Active
@mergify mergify bot merged commit c14f612 into main Oct 16, 2025
5 checks passed
@mergify mergify bot deleted the devs/sileht/scopes-summary/I60d468996a12845f9e5c770f529fd33515242124 branch October 16, 2025 14:49
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

4 participants