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

Add support for random string parameter plugin #23

Merged
merged 1 commit into from
Aug 11, 2014

Conversation

harcher81
Copy link
Contributor

No description provided.

@cloudbees-pull-request-builder

plugins » rebuild-plugin #36 SUCCESS
This pull request looks good

rsandell added a commit that referenced this pull request Aug 11, 2014
Add support for random string parameter plugin
@rsandell rsandell merged commit 278f249 into jenkinsci:master Aug 11, 2014
@oleg-nenashev
Copy link
Member

@rsandell
Robert, just for the future...
Rebuild Plugin provides the extension point, which allows users to define rebuild configurations for parameters in their plugins. It makes sense to advertise such approach instead of direct patches like this PR (just to keep the code consistent with explicit plugin dependencies).

@rsandell
Copy link
Member

aha, a feature that I've missed. apparently I haven't stayed fully up to date with the features, so I just assumed it was the usual approach.
I'll keep that in mind for the future.

@oleg-nenashev
Copy link
Member

This is quite new feature (thanks to @ikedam).
IMHO, it's great to know that someone else works on such kernel plugins :)

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.

4 participants