-
Notifications
You must be signed in to change notification settings - Fork 48
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
Upgrade to latest version of ZIO (2.0.8) #242
Conversation
f718b84
to
b5027db
Compare
b5027db
to
e8b7851
Compare
@williamareynolds Can you please sign the CLA? |
Done! I still haven't been able to run unit tests locally. I'm travelling and my laptop only has 8GB of RAM. I'll have access to a desktop soon, but it looks like the tests will run in CI. Edit: |
1baa027
to
ab77859
Compare
Alright, I think I've fixed the few little mistakes that were in here initially:
|
I think you need to update this line in
|
ab77859
to
a0990fa
Compare
@getquill/maintainers I think this should be ready to go now. It's been passing locally for me even without the heap size changes, but I think this out to do it in CI. |
Should we just update it to the freshly released 2.0.9? |
Any recent ZIO version will resolve this issue. |
I'm okay with either, but I'm thinking it'd be better to just leave it at 2.0.8 to get something working again. We can let a scala-steward PR do the update to 2.0.9. |
Fixes #217
Problem
The ZIO version used is currently set at 2.0.2. Using any version of ZIO later than 2.0.2 results in problems with the ZIO Environment.
Solution
Upgrade the ZIO dependencies to the latest version.
Notes
Additional notes.
Checklist
README.md
if applicable[WIP]
to the pull request title if it's work in progress@getquill/maintainers