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 to latest relation-graph. #1174

Merged
merged 1 commit into from
Dec 7, 2023
Merged

Update to latest relation-graph. #1174

merged 1 commit into from
Dec 7, 2023

Conversation

balhoff
Copy link
Contributor

@balhoff balhoff commented Dec 6, 2023

This is just an update to the latest relation-graph library, which itself is mainly an update to its dependencies. I'm hoping that updating to a newer ZIO dependency will resolve the mysterious test issues reported in #1166.

@balhoff balhoff requested a review from jamesaoverton December 6, 2023 22:50
@jamesaoverton
Copy link
Member

CI passed on the first try, and hopefully it will continue to! Much appreciated!

@jamesaoverton jamesaoverton merged commit 9dba31a into master Dec 7, 2023
3 checks passed
@jamesaoverton jamesaoverton deleted the refresh-rg branch December 7, 2023 21:09
@balhoff
Copy link
Contributor Author

balhoff commented Dec 7, 2023

I hope! Don't feel 100% confident; one of those ran for an hour and I restarted it. 😬 Let me know if you see any other issues.

@matentzn
Copy link
Contributor

matentzn commented Dec 7, 2023

one of those ran for an hour and I restarted it.

What do you mean exactly?

@balhoff
Copy link
Contributor Author

balhoff commented Dec 7, 2023

one of those ran for an hour and I restarted it.

What do you mean exactly?

The Java 17 CI job.

@jamesaoverton
Copy link
Member

We aren't paying for GitHub Actions in this repo. Won't they eventually timeout?

@balhoff
Copy link
Contributor Author

balhoff commented Dec 8, 2023

This one was in the test running phase for about an hour, so it seemed hung to me.

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.

3 participants