Skip to content
This repository has been archived by the owner on Aug 15, 2019. It is now read-only.

Update .travis.yml to send notifications on build failure. #1360

Merged
merged 2 commits into from
Oct 29, 2018
Merged

Conversation

nsthorat
Copy link
Contributor

@nsthorat nsthorat commented Oct 29, 2018

This change is Reviewable

@nsthorat nsthorat requested a review from dsmilkov October 29, 2018 16:00
Copy link
Contributor

@dsmilkov dsmilkov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewed 1 of 1 files at r1.
Reviewable status: 0 of 1 approvals obtained (waiting on @nsthorat and @dsmilkov)


.travis.yml, line 16 at r1 (raw file):

      - tensorflowjs-build@google.com
    on_success: never
    on_failure: always

add on_pull_requests: false to avoid getting spamed with every PR in progress that has failing unit tests

Copy link
Contributor Author

@nsthorat nsthorat left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Reviewable status: :shipit: complete! 1 of 1 approvals obtained


.travis.yml, line 16 at r1 (raw file):

Previously, dsmilkov (Daniel Smilkov) wrote…

add on_pull_requests: false to avoid getting spamed with every PR in progress that has failing unit tests

Done

@nsthorat nsthorat merged commit 5525357 into master Oct 29, 2018
@nsthorat nsthorat deleted the travis branch October 29, 2018 16:06
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants