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

Remove unneeded dependencies #214

Merged
merged 1 commit into from Sep 3, 2017

Conversation

Projects
None yet
2 participants
@Vlatombe
Member

Vlatombe commented Aug 30, 2017

There are a number of dependencies declared by the plugin which are actually not required, or that can be made optional.

Here is a recap of the list of changes

Moved to test scope:

  • workflow-cps
  • workflow-durable-task-step
  • workflow-support

Moved to optional

  • pipeline-model-extensions
    -> This requires to add a dependency to variant in order to leverage @OptionalExtension

Removed

  • pipeline-model-api as the plugin doesn't depend on it explicitly (only through pipeline-model-extensions)

@carlossg carlossg merged commit bc80951 into jenkinsci:master Sep 3, 2017

1 check passed

continuous-integration/jenkins/pr-merge This commit looks good
Details

@Vlatombe Vlatombe deleted the Vlatombe:make_declarative_optional branch Nov 23, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment