Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Support map[T]encoding.TextUnmarshaler #323

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

Al2Klimov
Copy link

No description provided.

@oxzi
Copy link
Contributor

oxzi commented Aug 14, 2024

Thanks for creating this PR. Just for the record, I have successfully tested it with my code over at Icinga/icinga-go-library#58 after removing the encoding.TextUnmarshaler for my custom map type.

Copy link

codecov bot commented Aug 24, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (aa50469) to head (c392b72).
Report is 21 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #323   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          499       516   +17     
=========================================
+ Hits           499       516   +17     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants