-
-
Notifications
You must be signed in to change notification settings - Fork 14
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
parse headers to struct using reflection tags #1
Conversation
Thanks for your contributor. Could you have time to add some tests for this feature? |
@mozillazg I've added a test derived from real world usage of Google Calendar's header based notification system. Thanks for reviewing the PR and considering its inclusion. |
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.
LGTM! Thanks!
@mozillazg thanks much! I'm working towards publishing the Google Calendar library. Do let me know if there's anything that I can do to get |
@bnfinet Just want to let you know, the new version of go-httpheader has been released today. BTW, I have updated some codes you can review these changes via v0.2.1...v0.3.0 . |
@mozillazg thank you! That's excellent test coverage. |
adds
decoder.go