Skip to content

Conversation

@dmarkham
Copy link
Contributor

@dmarkham dmarkham commented Apr 7, 2019

This is a starting point for the v1.4.0 release issue #1732.
This also merges in #1631 after fixing the conflicts.

@codecov
Copy link

codecov bot commented Apr 7, 2019

Codecov Report

Merging #1851 into master will increase coverage by 0.08%.
The diff coverage is 87.5%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1851      +/-   ##
==========================================
+ Coverage   98.65%   98.74%   +0.08%     
==========================================
  Files          41       38       -3     
  Lines        2149     2143       -6     
==========================================
- Hits         2120     2116       -4     
+ Misses         18       15       -3     
- Partials       11       12       +1
Impacted Files Coverage Δ
debug.go 95.91% <100%> (ø) ⬆️
render/redirect.go 100% <100%> (ø) ⬆️
context.go 98.36% <100%> (-0.02%) ⬇️
render/json.go 87.65% <100%> (+1.16%) ⬆️
response_writer.go 95.23% <50%> (-4.77%) ⬇️

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 202f8fc...45b9de9. Read the comment docs.

@dmarkham
Copy link
Contributor Author

dmarkham commented May 5, 2019

@appleboy @thinkerou I heard a rumor you guys might be able to do releases now. Can I do anything to make this PR more helpful to push this over the finish line!

@dmarkham dmarkham mentioned this pull request May 6, 2019
@appleboy
Copy link
Member

appleboy commented May 6, 2019

@dmarkham I have permission to create new tag now. I will review the PR asap

@appleboy appleboy added this to the 1.4 milestone May 6, 2019
@appleboy appleboy requested a review from thinkerou May 6, 2019 05:41
if values := req.MultipartForm.Value[key]; len(values) > 0 {
return values, true
}
}
Copy link
Member

Choose a reason for hiding this comment

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

Why remove this condition?

Copy link
Member

Choose a reason for hiding this comment

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

Add this code for support go1.6, please commit message: 51386de


if !exist && req.MultipartForm != nil && req.MultipartForm.File != nil {
dicts, exist = c.get(req.MultipartForm.Value, key)
}
Copy link
Member

Choose a reason for hiding this comment

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

same as above.

Copy link
Member

Choose a reason for hiding this comment

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

Add this code for support go1.6, please commit message: 51386de

@dmarkham
Copy link
Contributor Author

dmarkham commented May 7, 2019

🎆

@thinkerou thinkerou merged commit 094f9a9 into gin-gonic:master May 7, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants