Skip to content
This repository was archived by the owner on Jun 30, 2023. It is now read-only.

Inline Files.write() in GenApiConfigAction#175

Merged
gunsch merged 1 commit intocloudendpoints:masterfrom
nick-someone:patch-1
Dec 3, 2018
Merged

Inline Files.write() in GenApiConfigAction#175
gunsch merged 1 commit intocloudendpoints:masterfrom
nick-someone:patch-1

Conversation

@nick-someone
Copy link
Contributor

Files.write is deprecated and scheduled for removal. Changing this to use Files.asCharSink().write() is the same implementation, but is not deprecated.

Files.write is deprecated and scheduled for removal. Changing this to use Files.asCharSink().write() is the same implementation, but is not deprecated.
@codecov-io
Copy link

Codecov Report

Merging #175 into master will not change coverage.
The diff coverage is 0%.

Impacted file tree graph

@@            Coverage Diff            @@
##             master     #175   +/-   ##
=========================================
  Coverage     80.06%   80.06%           
  Complexity     1715     1715           
=========================================
  Files           159      159           
  Lines          5718     5718           
  Branches        754      754           
=========================================
  Hits           4578     4578           
  Misses          856      856           
  Partials        284      284
Impacted Files Coverage Δ Complexity Δ
...oogle/api/server/spi/tools/GenApiConfigAction.java 30.76% <0%> (ø) 2 <0> (ø) ⬇️

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 82710a4...4d28460. Read the comment docs.

@gunsch gunsch merged commit 4214e19 into cloudendpoints:master Dec 3, 2018
gunsch pushed a commit that referenced this pull request Dec 14, 2018
This is in reference to #175.
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.

4 participants