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

Make most things in core opaque to force helper functions and keep API clean #17

Open
2 tasks
mlutze opened this issue Jul 26, 2022 · 2 comments
Open
2 tasks

Comments

@mlutze
Copy link
Owner

mlutze commented Jul 26, 2022

@mlutze mlutze added this to the Community Build milestone Aug 5, 2022
@mlutze mlutze removed this from the Community Build milestone Sep 16, 2022
@JonathanStarup
Copy link
Collaborator

So what should be our local style guide? always use opaque? or always use opaque for pub-involved types?

I think we should make a style doc. for us but also as a collected document of feedback to flix

@mlutze
Copy link
Owner Author

mlutze commented Sep 24, 2022

Good idea.

I think the rule is that we always use opaque unless the shape of the ADT is actually part of the API. So for Move this is the case, but not for the stuff that we keep changing around.

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

No branches or pull requests

2 participants