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

pep8speaks is not getting updated #178

Closed
chkoar opened this issue Jun 29, 2020 · 2 comments
Closed

pep8speaks is not getting updated #178

chkoar opened this issue Jun 29, 2020 · 2 comments

Comments

@chkoar
Copy link

chkoar commented Jun 29, 2020

Hey,

Can you help us here? Are we missing something?
We made several changes but pep8speaks does not seem to respond anymore.

@OrkoHunter
Copy link
Collaborator

Hello!

Thank you for creating the issue. I manually resent a payload for this PR and the bot has updated its comment. I can see that the .pep8speaks.yml of the master branch is being used (88 chars max line length instead of 79).

Screenshot 2020-06-29 at 8 52 04 PM

I wonder there are two issues here -

  1. Perhaps the app has been uninstalled from the repo/org. You can take a look at the settings.
  2. The PR has touched 83 files and the bot needs to fetch each file and run the linter on them. Maybe it took a lot of time and the request timed out. Created Handle payloads with large number of files changed #179 to improve architecture.

Feel free to comment here if I need to force refresh the bot's comment. Thanks!

@chkoar
Copy link
Author

chkoar commented Jun 29, 2020

Great thanks.

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

No branches or pull requests

2 participants