Skip to content

Comments

Migrate to use Ruff for Linting and Formatting#141

Open
cboffa13 wants to merge 1 commit intooracle:mainfrom
cboffa13:cboffa/feature/ruff
Open

Migrate to use Ruff for Linting and Formatting#141
cboffa13 wants to merge 1 commit intooracle:mainfrom
cboffa13:cboffa/feature/ruff

Conversation

@cboffa13
Copy link
Member

@cboffa13 cboffa13 commented Feb 10, 2026

Description

Migrates the repo to use Ruff. https://docs.astral.sh/ruff/

Fixes #135

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

I re-ran all the commands for our CI / CD pipeline. The primary changes are formatting changes

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings
  • I have added tests that prove my fix is effective or that my feature works
  • New and existing unit tests pass locally with my changes
  • Any dependent changes have been merged and published in downstream modules

@oracle-contributor-agreement oracle-contributor-agreement bot added the OCA Verified All contributors have signed the Oracle Contributor Agreement. label Feb 10, 2026
@cboffa13 cboffa13 marked this pull request as draft February 10, 2026 17:22
@cboffa13 cboffa13 changed the title feat(ruff): replace tox with ruff linter Migrate to use Ruff for Linting and Formatting Feb 10, 2026
@cboffa13 cboffa13 force-pushed the cboffa/feature/ruff branch from 9198d46 to 5aed21e Compare February 10, 2026 17:26
@cboffa13 cboffa13 marked this pull request as ready for review February 10, 2026 17:31
".pytype",
".ruff_cache",
".svn",
".tox",
Copy link
Member

Choose a reason for hiding this comment

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

do we need this now that we're moved away from tox?

Copy link
Member

@gebhardtr gebhardtr left a comment

Choose a reason for hiding this comment

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

LGTM! minor comments re: tox artifacts; should we remove .tox from the .gitignore (and ruff.toml) now that it's gone?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

OCA Verified All contributors have signed the Oracle Contributor Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

use Ruff

2 participants