Skip to content

eli-366 addressing ruff failures with updated version of ruff#519

Merged
eddalmond1 merged 4 commits intomainfrom
chore/eja-eli-366-address-ruff-issues-with-version-bump
Jan 5, 2026
Merged

eli-366 addressing ruff failures with updated version of ruff#519
eddalmond1 merged 4 commits intomainfrom
chore/eja-eli-366-address-ruff-issues-with-version-bump

Conversation

@eddalmond1
Copy link
Copy Markdown
Collaborator

Description

Bumping ruff, addressing new linting failures detected and addressing them

Context

It's good to keep on top of ruff versions to keep our code nicely linted

Type of changes

  • Refactoring (non-breaking change)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would change existing functionality)
  • Bug fix (non-breaking change which fixes an issue)

Checklist

  • I am familiar with the contributing guidelines
  • I have followed the code style of the project
  • I have added tests to cover my changes
  • I have updated the documentation accordingly
  • This PR is a result of pair or mob programming

Sensitive Information Declaration

To ensure the utmost confidentiality and protect your and others privacy, we kindly ask you to NOT including PII (Personal Identifiable Information) / PID (Personal Identifiable Data) or any other sensitive data in this PR (Pull Request) and the codebase changes. We will remove any PR that do contain any sensitive information. We really appreciate your cooperation in this matter.

  • I confirm that neither PII/PID nor sensitive data are included in this PR and the codebase changes.

Karthikeyannhs
Karthikeyannhs previously approved these changes Jan 2, 2026
@eddalmond1 eddalmond1 force-pushed the chore/eja-eli-366-address-ruff-issues-with-version-bump branch from 1ae249e to 74b9325 Compare January 2, 2026 14:34
@eddalmond1 eddalmond1 merged commit 71b998e into main Jan 5, 2026
25 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.

3 participants