-
-
Notifications
You must be signed in to change notification settings - Fork 186
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
build(lint): switch over to ruff; add taplo #1444
Conversation
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #1444 +/- ##
=======================================
Coverage 97.46% 97.46%
=======================================
Files 48 48
Lines 4537 4537
=======================================
Hits 4422 4422
Misses 115 115
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
There's some Nix error, but I still don't know enough about Nix to say what's wrong. 🫣 And I wonder whether the Ruff pre-commit hook also checks/applies formatting because the hook spec looks a bit outdated. With a recent Ruff version, it's |
I see taplo in nix's configs, but nowhere else: what does it do? I guess it's used to format TOML files in VSCode? 🤔 |
Well, nix configs make the You have a brief explanation of how it works here. |
Ruff is faster, better integrated, easier to configure... Cool! Taplo also helps with toml maintenance.
Upstream issue is solved now.
Ruff is faster, better integrated, easier to configure... Cool!
Taplo also helps with toml maintenance.