You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
We run Ruff in pre-commit.ci, but also it turns out in many other CI jobs :)
I removed one extra run of Ruff in our CI last week or so: #567
But now reviewing #580 I see Ruff is running in our unit tests and integration tests. Ruff is relatively cheap and fast to run; are we using it to short-circuit running expensive tests?
This discussion was converted from issue #582 on December 10, 2024 19:46.
Heading
Bold
Italic
Quote
Code
Link
Numbered list
Unordered list
Task list
Attach files
Mention
Reference
Menu
reacted with thumbs up emoji reacted with thumbs down emoji reacted with laugh emoji reacted with hooray emoji reacted with confused emoji reacted with heart emoji reacted with rocket emoji reacted with eyes emoji
-
We run Ruff in pre-commit.ci, but also it turns out in many other CI jobs :)
I removed one extra run of Ruff in our CI last week or so: #567
But now reviewing #580 I see Ruff is running in our unit tests and integration tests. Ruff is relatively cheap and fast to run; are we using it to short-circuit running expensive tests?
Beta Was this translation helpful? Give feedback.
All reactions