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

[CI] Simplify CI setup using actions #5

Merged
merged 13 commits into from
Nov 22, 2022

Conversation

gergondet
Copy link
Contributor

Hi @mmurooka

This PR:

I could not fully test the PR due to the missing secret but afaik the eigen-lssol clone should work

@traversaro
Copy link
Contributor

Thanks @gergondet for taking this over!

CMakeLists.txt Show resolved Hide resolved
.github/workflows/ci.yaml Outdated Show resolved Hide resolved
@mmurooka
Copy link
Member

@gergondet @traversaro Thanks again for the PR.

I added some commits. CI is failing as below. Maybe this is temporary failure? I will merge this when CI passes.
https://github.com/gergondet/QpSolverCollection/actions/runs/3518986003

@gergondet
Copy link
Contributor Author

The CI failure is related to coin-or/qpOASES#136

I have pushed a temporary fix that should do the trick until this is merged

@gergondet
Copy link
Contributor Author

I have pushed a temporary fix that should do the trick until this is merged

It did the trick. However, I am not sure why the PR does not trigger a workflow :(

@mmurooka mmurooka merged commit abd3d6d into isri-aist:master Nov 22, 2022
@mmurooka
Copy link
Member

@gergondet Thank you. I don't know why CI is not triggered from PR, but CI in your fork passed, so I merged it.
https://github.com/gergondet/QpSolverCollection/actions/runs/3519359886

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.

None yet

3 participants