Skip to content

Conversation

@jd
Copy link
Member

@jd jd commented Oct 13, 2025

No description provided.

Change-Id: If4819dee49ec087a542087ab1b2fbde020824c55
Copilot AI review requested due to automatic review settings October 13, 2025 09:00
@mergify mergify bot deployed to Mergify Merge Protections October 13, 2025 09:00 Active
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 upgrades the ruff linter from version 0.13.1 to 0.14.0 and addresses linting issues that arose from the upgrade.

  • Updated ruff dependency from 0.13.1 to 0.14.0
  • Removed unused imports identified by the new linter version
  • Added noqa comment to suppress a new async-related warning

Reviewed Changes

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

File Description
pyproject.toml Updated ruff version from 0.13.1 to 0.14.0
mergify_cli/cli.py Removed unused import click.decorators
mergify_cli/init.py Removed unused import rich
mergify_cli/ci/junit_processing/junit.py Added noqa comment to suppress ASYNC240 warning for pathlib read operation

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 13, 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 13, 2025

🧪 CI Insights

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

🟢 All jobs passed!

But CI Insights is watching 👀

@mergify mergify bot merged commit 8ca8587 into main Oct 13, 2025
5 checks passed
@mergify mergify bot deleted the devs/jd/ruff/If4819dee49ec087a542087ab1b2fbde020824c55 branch October 13, 2025 09:21
@mergify mergify bot removed the queued label Oct 13, 2025
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.

3 participants