Skip to content

Conversation

seanpmorgan
Copy link
Member

Quick question: @ewilderj do you know if a travis server will be able to email that address without being in the google group?

@kyleabeauchamp
Copy link
Contributor

I checked the travis docs (https://docs.travis-ci.com/user/notifications/#configuring-email-notifications) and this seems consistent. LGTM once green!

kyleabeauchamp
kyleabeauchamp previously approved these changes Jun 12, 2019
@ewilderj
Copy link
Contributor

@seanpmorgan we ought to be able to directly add and whitelist the email address. hit me up once you know what the sender email address is

@seanpmorgan seanpmorgan changed the title Add notifications for failing tests/builds [WIP] Add notifications for failing tests/builds Jun 12, 2019
@seanpmorgan
Copy link
Member Author

Hmmm marking this as WIP because it looks like travis may not email addresses that aren't registered on github. Will look into this a little more later

@seanpmorgan
Copy link
Member Author

@ewilderj Could you please add builds@travis-ci.com to the google group (or whitelist). It seems we should be able to get this to work.
travis-ci/travis-ci#7169

I've temporarily added my email to the notifications so I can verify they're being sent at all.

@seanpmorgan seanpmorgan changed the title [WIP] Add notifications for failing tests/builds Add notifications for failing tests/builds Jun 13, 2019
@seanpmorgan seanpmorgan merged commit 829cd88 into tensorflow:master Jun 13, 2019
@seanpmorgan seanpmorgan deleted the add-notifications branch June 13, 2019 13:40
@ewilderj
Copy link
Contributor

@ewilderj Could you please add builds@travis-ci.com to the google group (or whitelist). It seems we should be able to get this to work.
travis-ci/travis-ci#7169

I've temporarily added my email to the notifications so I can verify they're being sent at all.

Done (for https://groups.google.com/a/tensorflow.org/forum/#!forum/addons-testing)

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants