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
Initial proposal for a Gin wrapper that provide graceful shutdown. #2
Initial proposal for a Gin wrapper that provide graceful shutdown. #2
Conversation
coverage.txt
Outdated
@@ -0,0 +1,65 @@ | |||
mode: atomic |
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 file to ignore list.
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.
Oops, good catch.
.gitignore
Outdated
@@ -22,3 +22,6 @@ _testmain.go | |||
*.exe | |||
*.test | |||
*.prof | |||
|
|||
coverage.out |
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.
coverage.*
.github/workflows/go.yml
Outdated
- name: Setup golangci-lint | ||
uses: golangci/golangci-lint-action@v3 | ||
with: | ||
version: v1.49.0 |
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.
latest
.github/workflows/go.yml
Outdated
strategy: | ||
matrix: | ||
os: [ubuntu-latest] | ||
go: [1.17, 1.18, 1.19, '1.20'] |
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.
remove 1.17
I think I have addressed all the first batch of comments. Is there anything I can do to help? |
@cedric-appdirect Thanks. |
The content of .github come from
gin-contrib/logger
and the test are inspired by gin test.