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

Pylint 1.6.3 compatibility by tseaver · Pull Request #1990 · googleapis/google-cloud-python · GitHub

Pylint 1.6.3 compatibility - #1990

Merged
tseaver merged 2 commits into
googleapis:masterfrom
tseaver:pylint-1.6.3-compat
Jul 18, 2016
Merged

Pylint 1.6.3 compatibility#1990
tseaver merged 2 commits into
googleapis:masterfrom
tseaver:pylint-1.6.3-compat

Conversation

tseaver commented Jul 18, 2016
edited
Loading

Copy link
Copy Markdown
Contributor

Fixes in pylint 1.6.2 and 1.6.3 mean we can test the files that caused it to barf in 1.6.0.

googlebot added the cla: yes This human has signed the Contributor License Agreement. label Jul 18, 2016

dhermes commented Jul 18, 2016

Copy link
Copy Markdown
Contributor

LGTM pending Travis

tseaver merged commit df47b57 into googleapis:master Jul 18, 2016
tseaver deleted the pylint-1.6.3-compat branch July 18, 2016 16:31
This was referenced Aug 3, 2016
parthea pushed a commit that referenced this pull request Mar 6, 2026
* fix: adding property setter for table constraints, #1990

* fix: adding unit test for to_api_repr() method

* fix: adding system test for bigquery table update

* fix: adding more test case for table constraints property

* fix: adding more test case for table constraints property

* fix: linting code

* fix: adding unit tests for test table, table constraint and foreign key

* fix: linting based on noxfile

* fix: linting based on noxfile

* fix: adding unit tests and system test

* fix: clearing lint error

* fix: adding table constraint eq unit tests

* fix: adding type to to_api_repr resource
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. testing

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants


Back | FazBrowse Home | New Git URL