Skip to content
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

Update docs on Context.Done(), Context.Deadline() and Context.Err() #2196

Merged
merged 4 commits into from Jan 7, 2020
Merged

Update docs on Context.Done(), Context.Deadline() and Context.Err() #2196

merged 4 commits into from Jan 7, 2020

Conversation

AstolfoKawaii
Copy link
Contributor

This issue has been discussed here #1452, here #1690, here #1736, here #2029, in my opinion, wether current behavior will be changed or not, current documentation doesn't match with real behavior, so it (documentation) has to be changed.
This is my view of what it should look like.

@codecov
Copy link

codecov bot commented Dec 29, 2019

Codecov Report

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

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #2196   +/-   ##
=======================================
  Coverage   98.51%   98.51%           
=======================================
  Files          41       41           
  Lines        2284     2284           
=======================================
  Hits         2250     2250           
  Misses         18       18           
  Partials       16       16
Impacted Files Coverage Δ
context.go 98.66% <ø> (ø) ⬆️

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 025950a...b1ac175. Read the comment docs.

@appleboy appleboy added this to the 1.6 milestone Jan 4, 2020
@thinkerou thinkerou merged commit 424e968 into gin-gonic:master Jan 7, 2020
ThomasObenaus pushed a commit to ThomasObenaus/gin that referenced this pull request Feb 19, 2020
byebyebruce pushed a commit to byebyebruce/gin that referenced this pull request Mar 25, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants