Skip to content
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

Two-directional transformations #547

Merged
merged 14 commits into from
Jun 25, 2024
Merged

Two-directional transformations #547

merged 14 commits into from
Jun 25, 2024

Conversation

MateuszKubuszok
Copy link
Member

@MateuszKubuszok MateuszKubuszok commented May 30, 2024

  • define Iso
  • define Codec
  • automatic derivation in AutoDerived companions after consideration - it is not needed
  • automatic derivation in chimney.auto after consideration - it is not needed
  • define
    • withFieldRenamed
  • extract total/partial from Iso/Codec
  • tests
  • documentation

@MateuszKubuszok MateuszKubuszok linked an issue May 30, 2024 that may be closed by this pull request
Copy link

codecov bot commented May 30, 2024

Codecov Report

Attention: Patch coverage is 97.46835% with 4 lines in your changes missing coverage. Please review.

Project coverage is 88.10%. Comparing base (a5c1e13) to head (bc4febd).
Report is 112 commits behind head on master.

Files with missing lines Patch % Lines
...nal/compiletime/derivation/codec/CodecMacros.scala 96.77% 1 Missing ⚠️
...nternal/compiletime/derivation/iso/IsoMacros.scala 96.77% 1 Missing ⚠️
...nal/compiletime/derivation/codec/CodecMacros.scala 96.77% 1 Missing ⚠️
...nternal/compiletime/derivation/iso/IsoMacros.scala 96.77% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master     #547      +/-   ##
==========================================
+ Coverage   87.98%   88.10%   +0.11%     
==========================================
  Files         138      148      +10     
  Lines        5536     5683     +147     
  Branches      470      475       +5     
==========================================
+ Hits         4871     5007     +136     
- Misses        665      676      +11     

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

@MateuszKubuszok MateuszKubuszok marked this pull request as ready for review June 25, 2024 15:20
@MateuszKubuszok MateuszKubuszok merged commit cc41c49 into master Jun 25, 2024
24 checks passed
@MateuszKubuszok MateuszKubuszok deleted the bi-transformers branch June 25, 2024 15:33
@nox213 nox213 mentioned this pull request Oct 17, 2024
10 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Bidirectional mapping
1 participant