gofrs / uuid

A UUID package for Go
MIT License
1.57k stars 110 forks source link

sql: assert UUID satisfies the driver.Valuer and sql.Scanner interfaces #113

Closed charlievieth closed 1 year ago

codecov-commenter commented 1 year ago

Codecov Report

Base: 100.00% // Head: 100.00% // No change to project coverage :thumbsup:

Coverage data is based on head (7eb9dd3) compared to base (ebca088). Patch has no changes to coverable lines.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## master #113 +/- ## ========================================= Coverage 100.00% 100.00% ========================================= Files 4 4 Lines 448 448 ========================================= Hits 448 448 ``` | [Impacted Files](https://codecov.io/gh/gofrs/uuid/pull/113?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=gofrs) | Coverage Δ | | |---|---|---| | [sql.go](https://codecov.io/gh/gofrs/uuid/pull/113?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=gofrs#diff-c3FsLmdv) | `100.00% <ø> (ø)` | | | [uuid.go](https://codecov.io/gh/gofrs/uuid/pull/113?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=gofrs#diff-dXVpZC5nbw==) | `100.00% <0.00%> (ø)` | | | [codec.go](https://codecov.io/gh/gofrs/uuid/pull/113?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=gofrs#diff-Y29kZWMuZ28=) | `100.00% <0.00%> (ø)` | | Help us with your feedback. Take ten seconds to tell us [how you rate us](https://about.codecov.io/nps?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=gofrs). Have a feature suggestion? [Share it here.](https://app.codecov.io/gh/feedback/?utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=gofrs)

:umbrella: View full report at Codecov.
:loudspeaker: Do you have feedback about the report comment? Let us know in this issue.

17twenty commented 1 year ago

Related to https://github.com/gofrs/uuid/pull/108 I assume?

charlievieth commented 1 year ago

Related to #108 I assume?

Yes, should have made that clearer.

cameracker commented 1 year ago

I'll give some other folks a chance to comment but i'll merge within the next few days. Thank you very much @charlievieth !

dylan-bourque commented 1 year ago

👍 from me