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

Set up pylint and clean code #19

Closed
wants to merge 15 commits into from
Closed

Set up pylint and clean code #19

wants to merge 15 commits into from

Conversation

y4izus
Copy link
Collaborator

@y4izus y4izus commented Jul 24, 2024

Summary

Details and comments

@Eric-Arellano
Copy link
Collaborator

I highly recommend Ruff instead. It's a much faster and more modern linter. Quite easy to use, as well. The Qiskit SDK team has been slowly migrating to it.

@y4izus
Copy link
Collaborator Author

y4izus commented Aug 1, 2024

I highly recommend Ruff instead. It's a much faster and more modern linter. Quite easy to use, as well. The Qiskit SDK team has been slowly migrating to it.

Thank you for the recommendation @Eric-Arellano !! as soon as I have some time, I will do the change to be align with the rest of the projects!

@y4izus
Copy link
Collaborator Author

y4izus commented Aug 23, 2024

Since we are gonna use Ruff instead of Pylint, I close this PR

@y4izus y4izus closed this Aug 23, 2024
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.

3 participants