Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix ruff-nbqa #815

Merged
merged 3 commits into from
Jun 29, 2024
Merged

Fix ruff-nbqa #815

merged 3 commits into from
Jun 29, 2024

Conversation

TylerJang27
Copy link
Collaborator

Ruff 0.5.0 deprecated running on files without the check subcommand. Fix our ruff-nbqa definition to support this and forward args correctly. Fixes failures in run

Copy link

trunk-io bot commented Jun 28, 2024

⏱️ 1h 17m total CI duration on this PR
Job Cumulative Duration Recent Runs
Linter Tests ubuntu-latest 26m 🟥🟩
Linter Tests macOS 23m 🟥🟩
Windows Linter Tests 8m 🟩🟩
Tool Tests (ubuntu-latest) 8m 🟩
Tool Tests (macOS) 5m 🟩
CodeQL-Build 3m 🟩🟩
Trunk Check runner [linux] 2m 🟩🟩
Action Tests 1m 🟩
Repo Tests / Plugin Tests 1m 🟩🟩
Detect changed files 14s 🟩🟩
Aggregate Test Results 4s 🟥🟩

settingsfeedbackdocs ⋅ learn more about trunk.io

@TylerJang27 TylerJang27 requested a review from det June 28, 2024 17:39
@TylerJang27 TylerJang27 marked this pull request as ready for review June 28, 2024 17:39
@TylerJang27 TylerJang27 enabled auto-merge (squash) June 28, 2024 22:36
@TylerJang27 TylerJang27 disabled auto-merge June 28, 2024 22:46
@TylerJang27 TylerJang27 merged commit 70f8284 into main Jun 29, 2024
15 checks passed
@TylerJang27 TylerJang27 deleted the tyler/fix-ruff-nbqa branch June 29, 2024 05:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants