Skip to content

ci: Set target version for ruff#1636

Merged
akoumpa merged 2 commits intomainfrom
donghyukc/lint_update
Apr 6, 2026
Merged

ci: Set target version for ruff#1636
akoumpa merged 2 commits intomainfrom
donghyukc/lint_update

Conversation

@thomasdhc
Copy link
Copy Markdown
Contributor

@thomasdhc thomasdhc commented Mar 31, 2026

What does this PR do ?

Ruff by default uses python 3.12, but support should be maintained starting with 3.10. Set ruff target version to 3.10

Changelog

  • Add specific line by line info of high level changes in this PR.

Before your PR is "Ready for review"

Pre checks:

  • Make sure you read and followed Contributor guidelines
  • Did you write any new necessary tests?
  • Did you add or update any necessary documentation?

If you haven't finished some of the above items you can still open "Draft" PR.

Additional Information

  • Related to # (issue)

Signed-off-by: Dong Hyuk Chang <9426164+thomasdhc@users.noreply.github.com>
@thomasdhc thomasdhc requested a review from a team as a code owner March 31, 2026 16:56
@copy-pr-bot
Copy link
Copy Markdown

copy-pr-bot bot commented Mar 31, 2026

This pull request requires additional validation before any workflows can run on NVIDIA's runners.

Pull request vetters can view their responsibilities here.

Contributors can view more details about this message here.

@thomasdhc thomasdhc requested a review from HuiyingLi March 31, 2026 16:56
@thomasdhc thomasdhc enabled auto-merge (squash) March 31, 2026 17:21
@thomasdhc
Copy link
Copy Markdown
Contributor Author

/ok to test 7335fd8

@thomasdhc
Copy link
Copy Markdown
Contributor Author

/ok to test ecf8b44

@akoumpa akoumpa disabled auto-merge April 6, 2026 00:08
@akoumpa akoumpa merged commit 91c6e41 into main Apr 6, 2026
56 checks passed
@akoumpa akoumpa deleted the donghyukc/lint_update branch April 6, 2026 00:08
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.

2 participants