-
Notifications
You must be signed in to change notification settings - Fork 8
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
Move to erlandono #81
base: master
Are you sure you want to change the base?
Conversation
schonfinkel
commented
Dec 4, 2024
•
edited
Loading
edited
- Closes Move to erlandono #80
d91151a
to
543fee0
Compare
543fee0
to
46b9ef6
Compare
%-type monad(_A) :: | ||
% {{ok, any(), any()}, select} | | ||
% {{ok, any()}, delete} | | ||
% {{ok, any()}, insert} | | ||
% {{ok, any(), any(), any()}, insert} | | ||
% {{ok, any()}, update} | | ||
% {{error, any()}, any()}. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could you elaborate why this would only make sense if we export it? I would like to keep it in order to keep it consistent on how the original erlando
does its monads.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I need to know which functions use this otherwise it's never used by the functions inside the module (it never typed anything) or outside it.
If you want to use outside it, we can export the type btw.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't follow why this needs to be used somewhere to be uncommented. My intention is to keep it in order to maintain the pattern created by erlando
. In this sense, keeping it in form of a comment, although non-ideal, is ok.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@EduardoLR10 then why not import monad.monad
? the lib actually exports the type.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I don't know, I actually haven't thought of that. Should we do that?