Skip to content

Remove Travis CI badging and update README and CONTRIBUTING. - #3038

Closed
daspecster wants to merge 1 commit into
googleapis:masterfrom
daspecster:remove-travisbadge
Closed

Remove Travis CI badging and update README and CONTRIBUTING.#3038
daspecster wants to merge 1 commit into
googleapis:masterfrom
daspecster:remove-travisbadge

Conversation

@daspecster

Copy link
Copy Markdown
Contributor

We haven't had a green travis build in a very long time.
Also, with the addition of Spanner, the master jobs reach the max time limit on Travis and just timeout anyway.

@googlebotgooglebot added the cla: yes This human has signed the Contributor License Agreement. label Feb 17, 2017
@dhermes

Copy link
Copy Markdown
Contributor

Also, with the addition of Spanner, the master jobs reach the max time limit on Travis and just timeout anyway.

This was true before Spanner.

I am 👍 on this but would rather we finish the migration from Travis to CircleCI first.

@daspecster

daspecster commented Feb 17, 2017

Copy link
Copy Markdown
ContributorAuthor

@dhermes what's missing in CiricleCI besides the PR builds only building/testing the changed services?

Since it builds everything each time, I think that overlaps the Travis coverage anyway.

@dhermes

Copy link
Copy Markdown
Contributor
  • the PR builds only building/testing the changed services
  • doing the same on merge builds (ci-diff-helper supports this)
  • making docs script less Travis specific
  • other parts of the iceberg?

@lukesneeringer

Copy link
Copy Markdown
Contributor

This is superceded by #3146.

Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla: yesThis human has signed the Contributor License Agreement.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@daspecster@dhermes@lukesneeringer@googlebot