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

template: add ruff as auto-formatter tool #220

Closed
xmnlab opened this issue Feb 23, 2024 · 3 comments · Fixed by #298
Closed

template: add ruff as auto-formatter tool #220

xmnlab opened this issue Feb 23, 2024 · 3 comments · Fixed by #298

Comments

@xmnlab
Copy link
Member

xmnlab commented Feb 23, 2024

It would be great to have ruff as a auto-formatter tool as an option (and maybe the default/first option as well)
although black is more popular than ruff, ruff is much more complete, faster, and it is less opinionated (more flexible) than black

@abhijeetSaroha
Copy link
Contributor

@xmnlab , Can you explain this issue more?

As of my understanding, ruff should be default option as auto-formatter tools.

It means, like in this image ruff should be automatically selected when this option comes.

Am i right?? or is it something else?

image

@xmnlab
Copy link
Member Author

xmnlab commented Mar 29, 2024

we should have two options for ruff: ruff (linter) and ruff (auto-formatter).

for OSL profile I am totally fine to remove black and keep just ruff (both ones) as default.

but for the base profile, we need to have all the options (black and both ruff options)

does it make sense?

Copy link

🎉 This issue has been resolved in version 0.9.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants