search

yahoojapan / authorization-proxy

Moved to https://github.com/AthenZ/authorization-proxy
https://github.com/AthenZ/authorization-proxy
Apache License 2.0
35 stars 9 forks source link

Upgrade go.mod #53

Closed kyfujisa closed 4 years ago

kyfujisa commented 4 years ago

upgrade to v2

WindzCUHK commented 4 years ago

need to update the pipeline at the same time like authorizer https://github.com/yahoojapan/athenz-authorizer/blob/master/.circleci/config.yml#L52

The current pipeline does NOT tag with the v prefix https://github.com/yahoojapan/authorization-proxy/blob/557e70aecf758440ebee570ab97cca9c6c5be2bf/.circleci/config.yml#L135

kyfujisa commented 4 years ago

@WindzCUHK oh OK.

kyfujisa commented 4 years ago

After this fix, I'll put a tag with a v-prefix on it.

codecov-io commented 4 years ago

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master      #53   +/-   ##
=======================================
  Coverage   86.73%   86.73%           
=======================================
  Files          14       14           
  Lines         641      641           
=======================================
  Hits          556      556           
  Misses         75       75           
  Partials       10       10
Impacted Files Coverage Δ
handler/handler.go 92.85% <ø> (ø)
handler/transport.go 100.00% <ø> (ø)
main.go 46.05% <ø> (ø)
router/debug_router.go 95.65% <ø> (ø)
router/debug_routes.go 88.88% <ø> (ø)
service/option.go 100.00% <ø> (ø)
service/server.go 97.63% <ø> (ø)
service/tls.go 78.94% <ø> (ø)
usecase/authz_proxyd.go 100.00% <ø> (ø)

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 557e70a...e4231b1. Read the comment docs.

kyfujisa commented 4 years ago

I need to include another modification, so I'll close it.