You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
ROM::Mapper still exposes old DSL that is far worse than what ROM::Transformer provides (via Dry::Transformer). Furthermore, we still use mapper builders with all the mapping options to build transformers, which is extra complexity that we don't need anymore.
This should be done prior #531 because adding Identity Map will be simpler with a simplified mapping layer.
TODO
Remove ROM::Mapper and its usage
Remove ROM::Header and its usage
Rename ROM::Transformer => ROM::Mapper
Refactor ROM::MapperCompiler so that it turns relation AST into a dry-transformer function w/o any intermediate objects as we have now
The text was updated successfully, but these errors were encountered:
@dede999 thanks for offering help. I gotta be honest though - this is one of the biggest and most difficult tasks scheduled for 6.0.0. I'm not sure if it's a good idea to tackle it as the first contribution. Obviously, nothing is blocking you from working on this one, I just want to make sure it's a good match for you. I'll be adding more issues and start using labels that will indicate the level of difficulty. This should make it simpler for people to pick up a suitable task.
ROM::Mapper
still exposes old DSL that is far worse than whatROM::Transformer
provides (viaDry::Transformer
). Furthermore, we still use mapper builders with all the mapping options to build transformers, which is extra complexity that we don't need anymore.This should be done prior #531 because adding Identity Map will be simpler with a simplified mapping layer.
TODO
ROM::Mapper
and its usageROM::Header
and its usageROM::Transformer
=>ROM::Mapper
ROM::MapperCompiler
so that it turns relation AST into a dry-transformer function w/o any intermediate objects as we have nowThe text was updated successfully, but these errors were encountered: