Skip to content
This repository was archived by the owner on Jul 16, 2021. It is now read-only.

Switches to negroni and gorilla/mux for routing#358

Merged
prydonius merged 1 commit intohelm:masterfrom
prydonius:negroni
Sep 21, 2017
Merged

Switches to negroni and gorilla/mux for routing#358
prydonius merged 1 commit intohelm:masterfrom
prydonius:negroni

Conversation

@prydonius
Copy link
Member

This removes the go-swagger server. Negroni and gorilla/mux will allow
for easier handling of middleware.

This removes the go-swagger server. Negroni and gorilla/mux will allow
for easier handling of middleware.
@codecov-io
Copy link

Codecov Report

Merging #358 into master will decrease coverage by 0.73%.
The diff coverage is 83.94%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #358      +/-   ##
==========================================
- Coverage   88.41%   87.67%   -0.74%     
==========================================
  Files          17       18       +1     
  Lines         820      844      +24     
==========================================
+ Hits          725      740      +15     
- Misses         61       68       +7     
- Partials       34       36       +2
Impacted Files Coverage Δ
src/api/middleware/incluster.go 100% <100%> (ø)
src/api/handlers/charts/charts.go 100% <100%> (ø) ⬆️
src/api/handlers/healthz.go 100% <100%> (ø) ⬆️
src/api/handlers/repos/repos.go 63.15% <58.33%> (-5.86%) ⬇️
src/api/handlers/releases/releases.go 96.87% <95%> (-3.13%) ⬇️

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 f51643e...49bb72e. Read the comment docs.

@prydonius prydonius merged commit f7c4045 into helm:master Sep 21, 2017
@prydonius prydonius deleted the negroni branch September 21, 2017 13:27
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.

2 participants