Skip to content

Conversation

@lukesneeringer
Copy link
Contributor

@lukesneeringer lukesneeringer commented Mar 10, 2017

I am not yet specifically asking for review on this (although @dhermes has already been providing feedback "as I go"). This is a PR at this point because I need to be able to thrash at CircleCI.


This is an attempt to fix the CI situation on this repository. In particular, this will hit the following goals:

  • Move to parallel builds on a per-API basis.
  • Move system tests into the individual API packages.
  • Eliminate Travis, move CircleCI to 2.0

In doing this, I made several changes:

  • Move from tox to nox (Python-based configuration is really useful for the multi-project situation)
  • Move from pylint to flake8 (pylint does not work on Python 3.6, has multiple difficult-to-silence false positives)

This is not done. In particular, several things need to be put back into place:

  • Automatic docs build.
  • Automatic PyPI releasing on appropriate tags.
  • Only testing APIs that have changed files.

@lukesneeringer lukesneeringer added the do not merge Indicates a pull request not ready for merge, due to either quality or timing. label Mar 10, 2017
@lukesneeringer lukesneeringer self-assigned this Mar 10, 2017
@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Mar 10, 2017
@lukesneeringer
Copy link
Contributor Author

Hey folks,
I am closing this, and going to make a google-cloud-python:ci branch, because I need to be able to test that system tests actually work.

@dhermes
Copy link
Contributor

dhermes commented Mar 15, 2017

Ha SGTM

@lukesneeringer lukesneeringer deleted the ci branch March 15, 2017 20:34
@lukesneeringer lukesneeringer mentioned this pull request Mar 15, 2017
3 tasks
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. do not merge Indicates a pull request not ready for merge, due to either quality or timing.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants