-
Notifications
You must be signed in to change notification settings - Fork 8.5k
v1.4.0 + #1631 (remove go1.6/go1,7 support) #1851
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Codecov Report
@@ 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
Continue to review full report at Codecov.
|
|
@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 I have permission to create new tag now. I will review the PR asap |
| if values := req.MultipartForm.Value[key]; len(values) > 0 { | ||
| return values, true | ||
| } | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why remove this condition?
There was a problem hiding this comment.
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) | ||
| } |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same as above.
There was a problem hiding this comment.
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
|
🎆 |
This is a starting point for the
v1.4.0release issue #1732.This also merges in #1631 after fixing the conflicts.