Skip to content

Conversation

@harshil21
Copy link
Member

@harshil21 harshil21 commented Sep 14, 2021

pylint used to run super quick, but a few months back it started to take 2+ mins on my machine. Apparently you can set the number of processors to speed it up. Setting it to 0 will auto detect no. of processors it can use. Much fast now! 😄

@harshil21 harshil21 added the ⚙️ tests affected functionality: tests label Sep 14, 2021
@harshil21 harshil21 added this to the v14 milestone Sep 14, 2021
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

Hey! Looks like you edited the (dev) requirements or the pre-commit hooks. I'm just a friendly reminder to keep the pre-commit hook versions in sync with the dev requirements and the additional dependencies for the hooks in sync with the requirements :)

files: ^(telegram|examples)/.*\.py$
args:
- --rcfile=setup.cfg
- --jobs=0
Copy link
Member

Choose a reason for hiding this comment

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

Would you mind adding a comment explaining what this does and/or linking to the pylint docs?

@Bibo-Joshi Bibo-Joshi merged commit b0ea2cc into v14 Sep 14, 2021
@Bibo-Joshi Bibo-Joshi deleted the fast-pylint branch September 14, 2021 17:56
Bibo-Joshi pushed a commit that referenced this pull request Sep 15, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Sep 16, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

⚙️ tests affected functionality: tests

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants