JuliaData / StructTypes.jl

Abstract definitions and convenience methods for describing, processing, and constructing Julia objects
https://juliadata.github.io/StructTypes.jl/stable/
MIT License
79 stars 22 forks source link

Remove keyword arguments before calling UUID constructor #86

Closed contradict closed 1 year ago

contradict commented 1 year ago

JSON3 passes keyword arguemnts to all constructors, need to remove them here since UUID doesn't take any.

codecov[bot] commented 1 year ago

Codecov Report

Base: 72.79% // Head: 72.86% // Increases project coverage by +0.07% :tada:

Coverage data is based on head (b9b61bf) compared to base (7318880). Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #86 +/- ## ========================================== + Coverage 72.79% 72.86% +0.07% ========================================== Files 2 2 Lines 386 387 +1 ========================================== + Hits 281 282 +1 Misses 105 105 ``` | [Impacted Files](https://codecov.io/gh/JuliaData/StructTypes.jl/pull/86?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JuliaData) | Coverage Δ | | |---|---|---| | [src/StructTypes.jl](https://codecov.io/gh/JuliaData/StructTypes.jl/pull/86/diff?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=JuliaData#diff-c3JjL1N0cnVjdFR5cGVzLmps) | `72.34% <100.00%> (+0.07%)` | :arrow_up: | 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=JuliaData). 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=JuliaData)

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