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

Remove unused dependencies from contrib, NIO and NIO examples #1333

Merged
merged 2 commits into from
Oct 25, 2016

Conversation

jean-philippe-martin
Copy link

As per issue #1332:

  • move mockito-core to test scope
  • change google-cloud-nio dependency to google-cloud-storage rather than google-cloud

The google-cloud-nio-example test passes, as expected.

$ java -cp google-cloud-contrib/google-cloud-nio/target/google-cloud-nio-0.4.1-SNAPSHOT-shaded.jar:google-cloud-contrib/google-cloud-nio-examples/target/google-cloud-nio-examples-0.4.1-SNAPSHOT.jar com.google.cloud.nio.examples.ListFilesystems
Installed filesystem providers:
  file
  jar
  gs

cc: @mziccard

@googlebot googlebot added the cla: yes This human has signed the Contributor License Agreement. label Oct 24, 2016
@coveralls
Copy link

Coverage Status

Coverage remained the same at 84.567% when pulling 55b534d on jean-philippe-martin:nio_deps into 953dd6f on GoogleCloudPlatform:master.

@@ -23,7 +23,7 @@
<dependencies>
<dependency>
<groupId>${project.groupId}</groupId>
<artifactId>google-cloud</artifactId>
<artifactId>google-cloud-storage</artifactId>

This comment was marked as spam.

This comment was marked as spam.

@@ -71,6 +71,7 @@
<groupId>org.mockito</groupId>
<artifactId>mockito-core</artifactId>
<version>1.9.5</version>
<scope>test</scope>

This comment was marked as spam.

This comment was marked as spam.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.005%) to 84.572% when pulling 706690f on jean-philippe-martin:nio_deps into 953dd6f on GoogleCloudPlatform:master.

@mziccard mziccard changed the title tighten dependencies Remove unused dependencies from contrib, NIO and NIO examples Oct 25, 2016
@mziccard mziccard merged commit fff9a44 into googleapis:master Oct 25, 2016
@mziccard
Copy link
Contributor

@jean-philippe-martin Thanks for this!

@jean-philippe-martin
Copy link
Author

Thanks for fixing the title, @mziccard !

github-actions bot pushed a commit that referenced this pull request Sep 15, 2022
…1575) (#1333)

Source-Link: googleapis/synthtool@2e9ac19
Post-Processor: gcr.io/cloud-devrel-public-resources/owlbot-java:latest@sha256:8175681a918181d306d9c370d3262f16b4c724cc73d74111b7d42fc985ca7f93
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes This human has signed the Contributor License Agreement.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants