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

[JENKINS-43693] improve multi-branch pipeline support #1187

Merged
merged 1 commit into from
Jun 19, 2019
Merged

[JENKINS-43693] improve multi-branch pipeline support #1187

merged 1 commit into from
Jun 19, 2019

Conversation

daspilker
Copy link
Member

No description provided.

@darxriggs
Copy link
Contributor

@daspilker other multi-branch sources like the one for Bitbucket are currently provided via dynamic DSL. Do you have an idea how to easily mark the id field as mandatory for these too?

@daspilker
Copy link
Member Author

@darxriggs Job DSL uses the same means as Pipeline to provide the dynamic DSL. See https://jenkins.io/doc/developer/plugin-development/pipeline-integration/#constructor-vs-setters.

So the plugins should declare the id field in a @DataBoundConstructor. The plugin can even provide documentation for the parameter, see https://github.com/jenkinsci/structs-plugin/blob/structs-parent-1.19/plugin/src/main/java/org/jenkinsci/plugins/structs/describable/DescribableParameter.java#L115.

Job DSL should not contain any special handling for plugins, this should be fixed in the plugins.

Can you try to get this fixed in the relevant plugins?

@daspilker daspilker merged commit 350005c into jenkinsci:master Jun 19, 2019
@daspilker daspilker deleted the JENKINS-43693 branch June 19, 2019 08:23
@darxriggs
Copy link
Contributor

darxriggs commented Jun 20, 2019

There was a change in the scm-api-plugin about 2 years ago in the opposite direction - see JENKINS-44891 and jenkinsci/scm-api-plugin#41.

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