FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

Update disabled messages so new pylint passes by daspecster · Pull Request #1967 · googleapis/google-cloud-python · GitHub

Update disabled messages so new pylint passes - #1967

Merged
daspecster merged 1 commit into
googleapis:masterfrom
daspecster:update-pylint-settings
Jul 8, 2016
Merged

Update disabled messages so new pylint passes#1967
daspecster merged 1 commit into
googleapis:masterfrom
daspecster:update-pylint-settings

Conversation

daspecster commented Jul 8, 2016
edited
Loading

Copy link
Copy Markdown
Contributor

Probably should discuss?

https://github.com/PyCQA/pylint/blob/master/ChangeLog

The docstring for a function or method should summarize its behavior and document its arguments, return value(s), side effects, exceptions raised, and restrictions on when it can be called (all if applicable). Optional arguments should be indicated. It should be documented whether keyword arguments are part of the interface.

From: https://www.python.org/dev/peps/pep-0257/

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 8, 2016
daspecster changed the title Update disabled messages from new pylint Update disabled messages so new pylint passes Jul 8, 2016

tseaver commented Jul 8, 2016

Copy link
Copy Markdown
Contributor

LGTM

tseaver commented Jul 8, 2016

Copy link
Copy Markdown
Contributor

Maybe add a hygeine issue which re-enables those options and fixes the errors?

Copy link
Copy Markdown
Contributor Author

Good idea!

daspecster merged commit 56b1a01 into googleapis:master Jul 8, 2016
This was referenced Aug 3, 2016
daspecster deleted the update-pylint-settings branch January 24, 2017 15:18
parthea added a commit that referenced this pull request Nov 24, 2025
Co-authored-by: Anthonios Partheniou <partheniou@google.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yes This human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL