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

Test scaffold should use Effect.Class.Console #328

Closed
justinwoo opened this issue Jul 22, 2019 · 1 comment

Comments

@justinwoo
Copy link
Collaborator

commented Jul 22, 2019

import Effect.Class.Console (log)

pros:

  • i can add Aff.launchAff_ without having to delete imports

cons:

  • more people need to learn the polymorphisms
@f-f

This comment has been minimized.

Copy link
Member

commented Jul 22, 2019

I don't really have an opinion on this, if you open a PR it's fine by me (also I think you're the original author of the scaffoldings, so go for it)

@justinwoo justinwoo referenced this issue Jul 24, 2019

Merged

Update test template #334

0 of 3 tasks complete

@mergify mergify bot closed this in #334 Jul 24, 2019

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.