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

Replaces black with ruff #94

Merged
merged 8 commits into from
Jul 10, 2024
Merged

Conversation

girol
Copy link
Contributor

@girol girol commented Jul 9, 2024

Extra

The Makefile changed significantly with this PR.

The idea was group similar commands by context. Some redundant commands were removed.

Also, a new recipe format was added.

girol added 4 commits July 9, 2024 11:11
- removes redundant commands
- groups multiple commands by context
- removes black formatter
- new command format
- Updates CONTRIBUTING.rst
@girol girol changed the title Replaces black to ruff Replaces black with ruff Jul 10, 2024
Copy link
Member

@zsinx6 zsinx6 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@girol Thanks for the PR!
There are just some small changes required

Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
girol and others added 4 commits July 10, 2024 15:14
@girol
Copy link
Contributor Author

girol commented Jul 10, 2024

@girol Thanks for the PR! There are just some small changes required

Fixes applied! Thanks for you review 🫶

@zsinx6 zsinx6 merged commit ff0f537 into grupy-sanca:main Jul 10, 2024
4 checks passed
@girol girol deleted the replace-black-to-ruff branch July 10, 2024 18:30
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