Skip to content

Conversation

@Narsil
Copy link
Contributor

@Narsil Narsil commented Jan 9, 2026

re-enable them manually).

Description

Here we're adding a new linter ruff check and disabling all checks that have violations (so we can add them one by one later with proper discussion about each).

Related Issue

Resolves Genesis-Embodied-AI/Genesis#

Motivation and Context

How Has This Been / Can This Be Tested?

Screenshots (if appropriate):

Checklist:

  • I read the CONTRIBUTING document.
  • I followed the Submitting Code Changes section of CONTRIBUTING document.
  • I tagged the title correctly (including BUG FIX/FEATURE/MISC/BREAKING)
  • I updated the documentation accordingly or no change is needed.
  • I tested my changes and added instructions on how to test it for reviewers.
  • I have added tests to cover my changes.
  • All new and existing tests passed.

@Narsil Narsil changed the base branch from nicolas/ruff to main January 9, 2026 17:02
@Narsil Narsil force-pushed the nicolas/ruff_check branch from 686bb54 to 2c031cf Compare January 9, 2026 17:02
@duburcqa duburcqa changed the title Adding ruff check (disabling currently existing issues so we can [MISC] Adding pre-commit ruff check. Jan 9, 2026
@duburcqa duburcqa merged commit 32cd539 into main Jan 9, 2026
20 of 21 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