Skip to content

jenkins: comment correct url - #180

Merged
maclover7 merged 1 commit into
nodejs:masterfrom
maclover7:jm-pipeline-2
Jun 9, 2018
Merged

maclover7 merged 1 commit into
nodejs:masterfrom
maclover7:jm-pipeline-2

Conversation

@maclover7

Copy link
Copy Markdown
Contributor

Was posting the API url for the build, now posts the
correct Blue Ocean url for the build.

Changes environment variable since we do not know what the url for the
build will be based on the API response.

Was posting the API url for the build, now posts the
correct Blue Ocean url for the build.

Changes environment variable since we do not know what the url for the
build will be based on the API response.
@maclover7
maclover7 requested a review from phillipj June 9, 2018 19:32
phillipj

This comment was marked as off-topic.

@maclover7

Copy link
Copy Markdown
Contributor Author

Thank you for all of your help @phillipj, I really appreciate it!

@maclover7
maclover7 merged commit 0c122d6 into nodejs:master Jun 9, 2018
@maclover7
maclover7 deleted the jm-pipeline-2 branch June 9, 2018 20:22
@maclover7

Copy link
Copy Markdown
Contributor Author

It Works: nodejs/node#21236 (comment) 🎉

maclover7 added a commit to nodejs/build that referenced this pull request Jun 9, 2018
Storing name of job, not URL anymore

PR-URL: #1328
Refs: https://github.com/nodejs-private/secrets/pull/21
Refs: nodejs/github-bot#180
Reviewed-By: Phillip Johnsen <johphi@gmail.com>
@phillipj

phillipj commented Jun 9, 2018

Copy link
Copy Markdown
Member

My pleasure, great having someone to work on the bot with 👍

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