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

Add more WriterT instances #620

Closed
19 tasks done
ceedubs opened this issue Nov 10, 2015 · 6 comments
Closed
19 tasks done

Add more WriterT instances #620

ceedubs opened this issue Nov 10, 2015 · 6 comments
Assignees

Comments

@ceedubs
Copy link
Contributor

ceedubs commented Nov 10, 2015

WriterT currently has Monad and Eq instances, but there are many more that it could have. Some examples that I think can be added are:

@ceedubs ceedubs added the ready label Nov 10, 2015
@ceedubs ceedubs self-assigned this Nov 11, 2015
@ceedubs
Copy link
Contributor Author

ceedubs commented Nov 11, 2015

I'll work on this.

ceedubs added a commit to ceedubs/cats that referenced this issue Nov 18, 2015
Knocks one item off of the checklist on typelevel#620.
This was referenced Nov 18, 2015
@akilegaspi
Copy link

Is this still in progress? I'm thinking of contributing for this enhancement.

@kailuowang
Copy link
Contributor

that'll be awesome. I updated the checklist, three instances still missing.

@akilegaspi
Copy link

Just a quick question.

Is the resolving type for the WriterT[F, L, A] after passing through extract is F[(L, A)]

@barambani
Copy link
Contributor

I would look into this one if that's ok.

@akilegaspi
Copy link

I think this should be closed since @barambani already finished it and it has been merged to master

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

6 participants