Skip to content

Conversation

@gpauloski
Copy link
Collaborator

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, PR/issue templates, repository configurations label Jan 23, 2024
@gpauloski gpauloski merged commit f4e34bc into main Jan 23, 2024
@gpauloski gpauloski deleted the flake-simplify branch January 23, 2024 16:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

development CI workflows, PR/issue templates, repository configurations

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants