Skip to content

Conversation

caspervonb
Copy link
Collaborator

@caspervonb caspervonb commented Oct 5, 2025

No description provided.

@caspervonb caspervonb requested a review from Copilot October 5, 2025 08:59
Copy link

@Copilot 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 switches the project from using multiple formatting tools (yapf, isort, flake8) to using ruff as a unified formatter and linter. The change consolidates the development dependencies and updates the GitHub Actions workflow to use ruff's formatting capabilities.

  • Removes yapf, isort, and flake8 from dependencies and replaces with ruff-only formatting
  • Configures ruff with specific formatting options (double quotes, space indentation)
  • Updates GitHub Actions workflow to use ruff format --check instead of separate yapf and isort checks

Reviewed Changes

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

File Description
pyproject.toml Removes yapf, isort, flake8 dependencies and configurations, adds ruff.format section
.github/workflows/check.yml Replaces yapf and isort checks with single ruff format check
Multiple Python files Automatic formatting changes applied by ruff (line length, quotes, spacing)

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

@caspervonb caspervonb marked this pull request as ready for review October 5, 2025 12:34
@caspervonb caspervonb changed the base branch from add-nats-client-package to main October 7, 2025 06:14
@caspervonb caspervonb changed the base branch from main to add-nats-client-package October 7, 2025 08:27
@caspervonb caspervonb force-pushed the add-nats-client-package branch 3 times, most recently from 7acfd1f to 2e13fe5 Compare October 7, 2025 15:02
@caspervonb caspervonb changed the base branch from add-nats-client-package to main October 8, 2025 19:20
@caspervonb caspervonb force-pushed the switch-to-ruff-for-formatting branch 2 times, most recently from 53a6763 to 72af19a Compare October 8, 2025 19:40
@caspervonb caspervonb force-pushed the switch-to-ruff-for-formatting branch from 72af19a to 766a012 Compare October 8, 2025 19:41
@caspervonb caspervonb merged commit 3fc3f79 into main Oct 8, 2025
56 checks passed
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.

1 participant