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

Extract reusable functions in inheritance #14

Merged
merged 13 commits into from
Nov 20, 2024
Merged

Extract reusable functions in inheritance #14

merged 13 commits into from
Nov 20, 2024

Conversation

nojaf
Copy link
Collaborator

@nojaf nojaf commented Nov 19, 2024

This uses the same

module Impl = (
  T: {
    type t
  },
) => {

trick like in #13.

So much code is deduplicated because of this.
@zth I strongly feel like we should do this.
Improving a binding in a base interface nicely ripples through to all the inheriting interfaces.

image

@nojaf
Copy link
Collaborator Author

nojaf commented Nov 20, 2024

So, I slept on this and maintenance-wise I feel like we should go for this.

@nojaf nojaf merged commit fb395c3 into main Nov 20, 2024
2 checks passed
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.

1 participant