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

Add conda-force-ci GitHub Action job #221

Merged
merged 6 commits into from
Mar 11, 2021
Merged

Conversation

traversaro
Copy link
Collaborator

Add GitHub Action job that compile the project using conda-forge-provided dependencies on Linux/macOS/Windows. The main advantage over apt-based workflow is that we always get the latest version of dependencies, easily detecting eventual regressions. Furthermore, no dependency needs to be compiled from source, making the GitHub Action simpler and more mantainable.

Similar PRs:

@traversaro
Copy link
Collaborator Author

The PR is WIP as the manif package is not available in conda-forge and will be added once conda-forge/staged-recipes#14142 is fixed.

@traversaro
Copy link
Collaborator Author

Eventually, this is the job that we can use for testing Python bindings also on Windows and macOS.

@traversaro traversaro changed the title [WIP] Add conda-force-ci GitHub Action job Add conda-force-ci GitHub Action job Mar 5, 2021
@traversaro
Copy link
Collaborator Author

There is still a missing dependency on lie-group-controllers as I still need to add it to robotology-staging channel. For what regards the changelog, this is not a user facing change, do you still want to have a line in the changelog for this @GiulioRomualdi ?

@GiulioRomualdi
Copy link
Member

No need to change the changelog :)

@traversaro
Copy link
Collaborator Author

lie-group-controllers added.

@traversaro
Copy link
Collaborator Author

@S-Dafarra @GiulioRomualdi let me know if there is any further action required on my side.

@S-Dafarra
Copy link
Member

Good to go for 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