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

Update dependencies, including upgrade to Cats Effect 3 #5

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

mariolyon
Copy link

@mariolyon mariolyon commented Apr 29, 2022

This PR updates dependencies, including Cats Effect 3 which has many changes from Cats Effect 2.
This was my guide: https://typelevel.org/cats-effect/docs/migration-guide.

The PR also include a scalafmt config copied from the typelevel/cats-effect repo, so there are formatting changes.

The logger is also being created wherever it is used, which is the safer and clearer way
(https://typelevel.org/log4cats/#examples).

Overall, it's runnable, and it works like the original.

@mariolyon mariolyon force-pushed the master branch 3 times, most recently from 26e05ad to ed54bae Compare April 29, 2022 14:41
@tpolecat
Copy link
Owner

Thanks! Looks like you're still working on this. Let me know when it's ready.

@mariolyon
Copy link
Author

mariolyon commented Apr 30, 2022 via email

@mariolyon
Copy link
Author

@tpolecat do you have time to review the PR? thanks

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.

2 participants