Skip to content

Conversation

@mhucka
Copy link
Collaborator

@mhucka mhucka commented Dec 31, 2025

This is a simple script to help contributors easily run pylint recursively on the right directories. It also passes --jobs=0 to make it as fast as possible by default.

This is a simple script to help contributors easily run pylint on the
right directories.
@github-actions github-actions bot added the size: S 10< lines changed <50 label Dec 31, 2025
Copy link
Contributor

@gemini-code-assist gemini-code-assist bot left a comment

Choose a reason for hiding this comment

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

Code Review

This pull request introduces a helpful script for running pylint. My review includes a few suggestions to align the script with the project's shell scripting style guide and to update a deprecated pylint option. Specifically, I've recommended changing the shebang to #!/bin/bash, adopting stricter shell options with set -euo pipefail, and removing the deprecated --recursive=y flag from the pylint command.

@mhucka mhucka added this pull request to the merge queue Jan 5, 2026
Merged via the queue into quantumlib:main with commit 40bd49e Jan 5, 2026
28 checks passed
@mhucka mhucka deleted the mh-pylint branch January 5, 2026 17:33
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

size: S 10< lines changed <50

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants