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

Adding back local dependencies to system test tox envs. by dhermes · Pull Request #2503 · googleapis/google-cloud-python · GitHub

Adding back local dependencies to system test tox envs. - #2503

Merged
dhermes merged 2 commits into
googleapis:masterfrom
dhermes:fix-2502
Oct 6, 2016
Merged

Adding back local dependencies to system test tox envs.#2503
dhermes merged 2 commits into
googleapis:masterfrom
dhermes:fix-2502

Conversation

dhermes commented Oct 6, 2016

Copy link
Copy Markdown
Contributor

Fixes #2502.

dhermes added the testing label Oct 6, 2016
googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 6, 2016

tseaver commented Oct 6, 2016

Copy link
Copy Markdown
Contributor

pycodestyle is no longer checking the whole tree (.git, .tox/ envs, etc.)

daspecster commented Oct 6, 2016
edited
Loading

Copy link
Copy Markdown
Contributor

@tseaver, do you think this is related to the strange behavior I saw in #2504?

dhermes mentioned this pull request Oct 6, 2016

dhermes commented Oct 6, 2016

Copy link
Copy Markdown
Contributor Author

@daspecster Indeed it's the same behavior (though it isn't strange).

@tseaver PTAL, fixed the problem in the second commit. As the commit message says, this was an unintended side-effect of #2486.

dhermes merged commit c441a39 into googleapis:master Oct 6, 2016
dhermes deleted the fix-2502 branch October 6, 2016 15:55
richkadel pushed a commit to richkadel/google-cloud-python that referenced this pull request May 6, 2017
Adding back local dependencies to system test tox envs.
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