Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

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

Merged
TravisEz13 merged 2 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
TravisEz13:AddTravisCIWebHookV2TravisEz13/PowerShell:AddTravisCIWebHookV2Copy head branch name to clipboard
Oct 20, 2017
Merged

Add code to send WebHook for Travis-CI Daily Build#5183
TravisEz13 merged 2 commits into
PowerShell:masterPowerShell/PowerShell:masterfrom
TravisEz13:AddTravisCIWebHookV2TravisEz13/PowerShell:AddTravisCIWebHookV2Copy head branch name to clipboard

Conversation

@TravisEz13

@TravisEz13 TravisEz13 commented Oct 20, 2017

Copy link
Copy Markdown
Member

Example output in Teams:
image

@daxian-dbw daxian-dbw left a comment

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.

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

Comment thread 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

Morty Proxy This is a proxified and sanitized view of the page, visit original site.