Skip to content
This repository has been archived by the owner on Jan 18, 2021. It is now read-only.

fix deprecation warning -> use onfigurableReport.setDestination(File) #817

Merged
merged 1 commit into from Apr 4, 2019

Conversation

epeee
Copy link
Contributor

@epeee epeee commented Apr 4, 2019

basically, getting rid of this warning

> Configure project :shipkit
The ConfigurableReport.setDestination(Object) method has been deprecated and is scheduled to be removed in Gradle 5.0. Please use the method ConfigurableReport.setDestination(File) instead.
        at shipkit_2cr6fz4tp44wkkz1iroi80u1s$_run_closure9$_closure18.doCall(/home/ep/projects/github/mockito-project/mockito-release-tools/subprojects/shipkit/shipkit.gradle:103)

Copy link
Contributor

@mstachniuk mstachniuk left a comment

Choose a reason for hiding this comment

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

👍 Tried to fix this same right now, but your code is more elegant

@epeee epeee merged commit cfd87f2 into master Apr 4, 2019
@codecov
Copy link

codecov bot commented Apr 4, 2019

Codecov Report

Merging #817 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #817   +/-   ##
=========================================
  Coverage     94.25%   94.25%           
  Complexity      475      475           
=========================================
  Files           156      156           
  Lines          2244     2244           
  Branches         77       77           
=========================================
  Hits           2115     2115           
  Misses           61       61           
  Partials         68       68

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5b08153...5b674ff. Read the comment docs.

@epeee epeee deleted the ep2 branch April 4, 2019 17:44
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants