KIT-MRT / arbitration_graphs

Hierarchical behavior models for complex decision-making and behavior generation in robotics
https://kit-mrt.github.io/arbitration_graphs/
MIT License
15 stars 0 forks source link

Add GitHub actions #61

Closed ll-nick closed 2 weeks ago

ll-nick commented 1 month ago

See #59 for details.

This adds automated workflows to

ll-nick commented 2 weeks ago

Changed the base to packaging_with_cpack since we need those changes for the release pipeline.

ll-nick commented 2 weeks ago

Alright @orzechow , this one is ready for review. It should be very analogous to the util_caching pipelines now. Feel free to push your release-test-pipeline onto this branch if we want to do that in one go.

orzechow commented 2 weeks ago

Alright @orzechow , this one is ready for review. It should be very analogous to the util_caching pipelines now. Feel free to push your release-test-pipeline onto this branch if we want to do that in one go.

Thanks!

One quick remark: the dockerfile is missing the release_builder stage etc.

Yeah, I'll add the release test pipeline after finishing the cpack PR

ll-nick commented 2 weeks ago

You're right. I think it's easier if one person just does it. I have moved the PR back to draft to wait until you finished the cpack PR.

I will then rebase this PR, add the Docker stage and release test pipeline and ping you once the PR is actually done.

ll-nick commented 2 weeks ago

Rebased on main

ll-nick commented 2 weeks ago

Ok @orzechow this one should truly be ready now.

See the edited PR description for all the things that are now going on the workflows.

I tested the workflow on a test branch where I didn't need a PR to trigger the workflow, see 82b31a0d2e551b35fb531e5b0fb21f45be565fc6. Other than that, the branch is identical to this one and the last run was succesful.

A couple of noteworthy changes I made since we last talked about this:

Before merging, I'll delete the tag, release and docker images created by the test branch. I will then merge this as a #minor release.