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 GitHub workflow for Windows CI #109

Open
wants to merge 8 commits into
base: dashing
Choose a base branch
from

Conversation

jacobperron
Copy link
Contributor

This is an attempt at adding Windows CI via a GitHub workflow.

@jacobperron
Copy link
Contributor Author

The workflow is refusing to run: https://github.com/ros2-java/ros2_java/actions/runs/123058318

After some searching, it sounds like we need to edit the settings for this repository (or the ros2-java org) to allow third-party GitHub actions. @esteve I don't have admin privileges, are you able to look into it?

Signed-off-by: Jacob Perron <jacob@openrobotics.org>
@esteve esteve force-pushed the jacob/windows_workflow branch from 998300f to 7701ed5 Compare June 6, 2020 07:06
@esteve
Copy link
Member

esteve commented Jun 6, 2020

The Windows workflow is now running https://github.com/ros2-java/ros2_java/pull/109/checks?check_run_id=744698928

I had to push to your branch to trigger the CI, but there are no changes (I just force-pushed to change the tip of your branch). You may need to pull --rebase your local branch, though.

@jacobperron
Copy link
Contributor Author

@esteve Thanks 🙇

For Linux, it looks like a new flake8 error upstream. I think ament/ament_lint#252 should resolve the issue ultimately.

I'll continue looking into what's up with the Windows build.

Signed-off-by: Jacob Perron <jacob@openrobotics.org>
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
This gives more control over which linters are run. For example, we don't need to depend on Python linters for many packages.
This commit also worksaround an upstream issue with CI, where ament_cmake_flake8 is currently failing with a Dashing installation
if flake8 is installed from pip. See ament/ament_lint#252 for a fix upstream.

Signed-off-by: Jacob Perron <jacob@openrobotics.org>
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
Signed-off-by: Jacob Perron <jacob@openrobotics.org>
@jacobperron
Copy link
Contributor Author

I've gotten one step further with Windows CI. Looks like I've run into a known issue with the ros-tooling/setup-ros action (ros-tooling/action-ros-ci#79).

* Fix Windows problem not finding ASIO.

It basically needs an upgrade to setup-ros and action-ros-ci.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>

* Fixes from review.

Signed-off-by: Chris Lalancette <clalancette@openrobotics.org>
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