Skip to content

[All] Remove Jackson from cucumber-jvm parent pom #1900

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

Merged
merged 1 commit into from
Feb 21, 2020

Conversation

mpkorstanje
Copy link
Contributor

@mpkorstanje mpkorstanje commented Feb 21, 2020

The cucumber-jvm parent pom included jackson-databind in its dependency
management section. Because this is a test only dependency it was marked as
test scoped.

While not intended to function as a bill of materials people were and perhaps
still are using cucumber-jvm in this fashion. Removing jackson-databind
from the dependency management ensures that cucumber-jvm only manages
cucumber and test related dependencies.

While not safe, this does provide a modicum of safety.

Types of changes

  • Bug fix (non-breaking change which fixes an issue).
  • New feature (non-breaking change which adds functionality).
  • Breaking change (fix or feature that would cause existing functionality to not work as expected).

The `cucumber-jvm` parent pom included `jackson-databind` in its dependency
management section. Because this is a test only dependency it was marked as
test scoped.

While not intended to function as a bill of materials people were and perhaps
still are using `cucumber-jvm` in this fashion. Removing `jackson-databind`
from the dependency management ensures that `cucumber-jvm` only manages
cucumber and test related dependencies.
@mpkorstanje mpkorstanje added this to the 5.x.x milestone Feb 21, 2020
@coveralls
Copy link

Coverage Status

Coverage remained the same at 83.02% when pulling 14cb06f on push-jackson-down-to-modules into 368aed3 on master.

@mpkorstanje mpkorstanje merged commit 0386fe7 into master Feb 21, 2020
@mpkorstanje mpkorstanje deleted the push-jackson-down-to-modules branch February 21, 2020 18:22
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.

3 participants