Skip to content

Add symbiotic runner#426

Merged
redianthus merged 2 commits intoOCamlPro:mainfrom
filipeom:symbiotic-runner
May 7, 2025
Merged

Add symbiotic runner#426
redianthus merged 2 commits intoOCamlPro:mainfrom
filipeom:symbiotic-runner

Conversation

@filipeom
Copy link
Collaborator

Symbiotic stuff if you want to add it

@filipeom filipeom force-pushed the symbiotic-runner branch 2 times, most recently from d81228d to ff98974 Compare August 30, 2024 08:38
@filipeom
Copy link
Collaborator Author

filipeom commented May 6, 2025

I don't see us needing this in the future right? Maybe I can close it

@redianthus
Copy link
Member

I think it would still be interesting to have it. What is missing currently? I may be able to take care of it if you don't have time to spend on it

@filipeom
Copy link
Collaborator Author

filipeom commented May 6, 2025

Rebase it. I think if the CI is good it's probably good to go. I ran symbotic for the paper with this using the testcomp runner

@redianthus
Copy link
Member

Oh, OK! Great, thanks for rebasing! Can you confirm that dune b @all is working locally? (I'm not sure the CI is testing it) Then I can merge :)

@filipeom
Copy link
Collaborator Author

filipeom commented May 6, 2025

Oh, OK! Great, thanks for rebasing! Can you confirm that dune b @all is working locally? (I'm not sure the CI is testing it) Then I can merge :)

It works on my machine (famous last words 😅)

@redianthus
Copy link
Member

Hahaha, great, thanks!

@redianthus redianthus merged commit 6ac7657 into OCamlPro:main May 7, 2025
5 of 6 checks passed
@filipeom filipeom deleted the symbiotic-runner branch May 7, 2025 17:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants