Apply PECS where it is missing

roookeee / datus

datus enables you to define a conversion process between two data structures in a fluent functional API

MIT License

41 stars 5 forks source link

Apply PECS where it is missing #16

Closed roookeee closed 5 years ago

roookeee commented 5 years ago

As a library datus should follow PECS. Most parts of datus are already PECS compliant but some functions that were added just before releasing 1.0 don't comply yet (or they have just slipped my attention).

I will categorize this issue as semver-minor because applying PECS is backwards compatible and not being able to use functions/predicates/etc. that would work with PECS is more of a bug than a new feature.

[x] Mapper interface

roookeee commented 5 years ago

Done with the Mapper interface in branch issue-16

roookeee commented 5 years ago

I will exclude the Datus helper class as it would be pretty bizarre to run in any PECS related issues there

roookeee commented 5 years ago

I have triple checked every class now, let's hope I found everything - will merge this now