Skip to content
This repository was archived by the owner on Oct 23, 2025. It is now read-only.

Conversation

@jmcs
Copy link
Member

@jmcs jmcs commented Sep 30, 2016

  • Docstrings;
  • More specific variable names;
  • Reduced some code repetition;

There are still a couple of warnings because sentry is constant but lower case, the catch-all except and the typing imports.

@coveralls
Copy link

coveralls commented Sep 30, 2016

Coverage Status

Coverage remained the same at 89.753% when pulling 40609d4 on pylint-friday-4 into d5be62f on master.

@rafaelcaricio
Copy link
Contributor

👍

@hjacobs hjacobs merged commit 294e2d5 into master Oct 1, 2016
@rafaelcaricio rafaelcaricio deleted the pylint-friday-4 branch October 4, 2016 09:22
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

9 participants