Skip to content

Add code to send WebHook for Travis-CI Daily Build#5183

Merged
TravisEz13 merged 2 commits intoPowerShell:masterfrom
TravisEz13:AddTravisCIWebHookV2
Oct 20, 2017
Merged

Add code to send WebHook for Travis-CI Daily Build#5183
TravisEz13 merged 2 commits intoPowerShell:masterfrom
TravisEz13:AddTravisCIWebHookV2

Conversation

@TravisEz13
Copy link
Copy Markdown
Member

@TravisEz13 TravisEz13 commented Oct 20, 2017

Example output in Teams:
image

Copy link
Copy Markdown
Member

@daxian-dbw daxian-dbw left a comment

Choose a reason for hiding this comment

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

Looks good except for a minor comment.
A question: where do we set the $webhook environment variable?

tools/travis.ps1 Outdated
}
catch {
Write-Warning "Could not send webhook: $_"

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Remove the extra line?

@TravisEz13
Copy link
Copy Markdown
Member Author

The WebhookURL environment variable will be set in travis-ci.org itself. the same way as the secrets to upload artifacts.

@TravisEz13 TravisEz13 self-assigned this Oct 20, 2017
@TravisEz13 TravisEz13 merged commit 13eb283 into PowerShell:master Oct 20, 2017
@TravisEz13 TravisEz13 deleted the AddTravisCIWebHookV2 branch October 20, 2017 22:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants