-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Package frenetic.5.0.5 #19561
Package frenetic.5.0.5 #19561
Conversation
All builds are failing with
|
So sorry. I seem to have had an outdated version of |
@jnfoster what is the status of this, shuld we close it? |
@jnfoster which version of ppxlib did you have installed? We can just add an upper bound on the package description here and merge this. A future revision can include the latest ASTs. |
26ac61d
to
614a4fd
Compare
Hi! I removed some fragile and hard to maintain PPX code that was rarely used, and updated dependencies in the |
614a4fd
to
7f962da
Compare
🙏 You are very kind to review and repair the disorganized commits from a busy professor. |
Many of the CI builds are failing because of an MD5 sum. I created my PR with |
The 5.0.5 tag used for the archive has been force-pushed a couple of hours ago after your last change here, so the checksum is different from the one submitted by opam publish. Could you either force-push it back to the original or update the checksum in this PR? |
1540d94
to
364dfef
Compare
364dfef
to
86395b9
Compare
Seems like the CI is stalled only on the MacOS / Homebrew jobs.
Is there anything that can be done to get this unstuck? Did the project run out of resources for the week/month/year? |
Yeah, the macOS runners are stuck. We can merge anyway i think. Thanks! |
Woohoo! Thank you again for patiently reviewing this and helping me get everything in order. It's great to have Frenetic building in opam again. |
frenetic.5.0.5
The Frenetic Programming Language and Runtime System
🐫 Pull-request generated by opam-publish v2.1.0