Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Expand parameters supplied by trigger.getBuild*Message() #331

Merged
merged 1 commit into from Oct 25, 2017

Conversation

Projects
None yet
2 participants
@GLundh
Copy link
Member

commented Oct 24, 2017

Example use-case: An URL is generated to a staging environment based
on the commit. We want to send back that url to the user:

"This change is staged at http://staging-host/$GERRIT_PATCHSET_REVISION"

Expand parameters supplied by trigger.getBuild*Message()
Example use-case: An URL is generated to a staging environment based
on the commit. We want to send back that url to the user:

"This change is staged at http://staging-host/$GERRIT_PATCHSET_REVISION"
@GLundh

This comment has been minimized.

Copy link
Member Author

commented Oct 24, 2017

Thanks Bobby! Am I free to merge, or do you want additional reviewers on it?

@rsandell

This comment has been minimized.

Copy link
Member

commented Oct 25, 2017

I didn't merge because I was waiting for Jenkins to return the build result :)

@rsandell rsandell merged commit 16017f6 into jenkinsci:master Oct 25, 2017

1 check failed

continuous-integration/jenkins/pr-merge This commit cannot be built
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
You can’t perform that action at this time.