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

Add Tuple transformations section to the ducktape comparison #679

Merged

Conversation

arainko
Copy link
Contributor

@arainko arainko commented Jan 26, 2025

Adding in comparisons between tuple transformations and more up-to-date usage of Mode[F] (i.e. using .locally with a local Mode as opposed to a single toplevel one)

Screenshots of the new section:
Screenshot from 2025-01-26 13-20-27
Screenshot from 2025-01-26 13-20-35

@@ -1934,8 +2034,6 @@ Since Ducktape is inspired by Chimney, there is a huge overlap in functionality.
* customizing the [field-](supported-transformations.md#customizing-field-name-matching) and
[subtype-name](supported-transformations.md#customizing-subtype-name-matching) matching methods
* [sharing flags overrides between all derivations in the same scope](cookbook.md#reusing-the-flags-for-several-transformationspatchings)
* [smart constructors](supported-transformations.md#types-with-manually-provided-constructors), not only custom
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dropping this point since ducktape also supports smart cosntructors (https://github.com/arainko/ducktape/blob/456df5656e223ebab9bcc044f65230481591b6ec/ducktape/src/main/scala/io/github/arainko/ducktape/syntax.scala#L107) - I'll have a follow up PR that fills this in the docs as well 😄

@MateuszKubuszok MateuszKubuszok merged commit 92e8044 into scalalandio:master Jan 27, 2025
22 checks passed
@MateuszKubuszok
Copy link
Member

Thanks!

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.

2 participants