Skip to content

Change slf4j-simple to a test runtime dependency#201

Merged
Ben Fradet (BenFradet) merged 2 commits into
snowplow:developfrom
dnedev:develop
Aug 7, 2018
Merged

Change slf4j-simple to a test runtime dependency#201
Ben Fradet (BenFradet) merged 2 commits into
snowplow:developfrom
dnedev:develop

Conversation

@dnedev

Copy link
Copy Markdown
Contributor
  • Add compile dependency on slf4j-api.
  • Change slf4j-simple to only a test runtime one.

This should resolve #188

@dnedev

Copy link
Copy Markdown
Contributor Author

If you are open to it, and it introduces no breaking changes, we can also upgrade the SLF4J to the latest 1.7.x version - 1.7.25

@snowplowcla

Copy link
Copy Markdown

Thanks for your pull request. Is this your first contribution to a Snowplow open source project? Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

📝 Please visit https://github.com/snowplow/snowplow/wiki/CLA to learn more and sign.

Once you've signed, please reply here (e.g. I signed it!) and we'll verify. Thanks.

@snowplowcla Snowplow CLA bot (snowplowcla) added the cla:no [Auto generated] Snowplow Contributor License Agreement has not been signed. label Aug 6, 2018
@dnedev

Copy link
Copy Markdown
Contributor Author

Signed the CLA, thanks

@snowplowcla

Copy link
Copy Markdown

Confirmed! Dimitar Nedev (@dnedev) has signed the Individual Contributor License Agreement. Thanks so much

@snowplowcla Snowplow CLA bot (snowplowcla) added cla:yes [Auto generated] Snowplow Contributor License Agreement has been signed. and removed cla:no [Auto generated] Snowplow Contributor License Agreement has not been signed. labels Aug 7, 2018

@BenFradet Ben Fradet (BenFradet) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

@BenFradet Ben Fradet (BenFradet) merged commit 56f00c2 into snowplow:develop Aug 7, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

cla:yes [Auto generated] Snowplow Contributor License Agreement has been signed.

Development

Successfully merging this pull request may close these issues.

Use org.slf4j:slf4j-api as compile dependency, not org.slf4j:slf4j-simple

3 participants