Skip to content

Conversation

@gpauloski
Copy link
Contributor

Description

Enable and fix ruff C90, SIM, and YTT lints (mccabe, flake8-simplify, and flake8-2020).

Fixes N/A

Type of Change

  • Breaking Change (fix or enhancement which changes existing semantics of the public interface)
  • Enhancement (new features or improvements to existing functionality)
  • Bug (fixes for a bug or issue)
  • Internal (refactoring, style changes, testing, optimizations)
  • Documentation update (changes to documentation or examples)
  • Package (dependencies, versions, package metadata)
  • Development (CI workflows, pre-commit, linters, templates)
  • Security (security related changes)

Testing

Ruff passes.

Pull Request Checklist

Please confirm the PR meets the following requirements.

  • Tags added to PR (e.g., breaking, bug, enhancement, internal, documentation, package, development, security).
  • Code changes pass pre-commit (e.g., mypy, ruff, etc.).
  • Tests have been added to show the fix is effective or that the new feature works.
  • New and existing unit tests pass locally with the changes.
  • Docs have been updated and reviewed if relevant.

@gpauloski gpauloski added the development CI workflows, pre-commit, linters, templates label Jan 23, 2024
@gpauloski gpauloski merged commit 06c3554 into main Jan 23, 2024
@gpauloski gpauloski deleted the ruff-lint-rules branch January 23, 2024 16:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

development CI workflows, pre-commit, linters, templates

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants