Skip to content

Conversation

@sileht
Copy link
Member

@sileht sileht commented Oct 14, 2025

It makes more obvious that user has to do other stuffs manually.

Copilot AI review requested due to automatic review settings October 14, 2025 08:52
@mergify mergify bot had a problem deploying to Mergify Merge Protections October 14, 2025 08:52 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 renames the SourceOther class and related references to SourceManual to improve clarity for users about manual scope configuration requirements.

  • Renamed SourceOther class to SourceManual with corresponding field name change
  • Updated type annotations and error messages to use the new naming
  • Simplified error message format for better readability

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

@mergify
Copy link
Contributor

mergify bot commented Oct 14, 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 14, 2025

🧪 CI Insights

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

🟢 All jobs passed!

But CI Insights is watching 👀

jd
jd previously approved these changes Oct 14, 2025
It makes more obvious that user has to do other stuffs manually.

Change-Id: Ice3f956e29313e5f2f7552d2fb229bdfe7a9d5b3
@sileht sileht force-pushed the devs/sileht/cli2-other/Ice3f956e29313e5f2f7552d2fb229bdfe7a9d5b3 branch from 4433789 to c0ce9ab Compare October 14, 2025 11:28
@mergify mergify bot dismissed jd’s stale review October 14, 2025 11:29

Pull request has been modified.

@mergify mergify bot deployed to Mergify Merge Protections October 14, 2025 11:29 Active
@mergify mergify bot requested a review from a team October 14, 2025 11:30
@mergify mergify bot merged commit 9e00961 into main Oct 14, 2025
5 checks passed
@mergify mergify bot deleted the devs/sileht/cli2-other/Ice3f956e29313e5f2f7552d2fb229bdfe7a9d5b3 branch October 14, 2025 12:02
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