ar-tools / ar_tools

AR Marker tools for ROS
31 stars 51 forks source link

Enable travis #23

Open dlaz opened 8 years ago

dlaz commented 8 years ago

Could someone who has the appropriate privileges enable travis? I created a .travis.yml, but lack the privileges to enable it.

130s commented 6 years ago

+1

Downstream repo that uses packages from this repo faced a build issue (https://github.com/Nishida-Lab/motoman_project/issues/117, in Japanese), which could have been avoided if Travis was enabled (and resolved potential build issue).

MoriKen254 commented 6 years ago

I personally modified this package so as to make it pass industrial_ci 's build and test processes. You can see the repository on this link.

You'll see the travis CI status is finely built.

image

Summary Figure

The summary is as bellow.

1

2

Points

Request

I know this modification would give kinda huge impact considering pkgs previously using the conventional ar_tools.

However, this could also compensate to next pkgs willing to use ar_tools.

If you would somewhat agree with my idea, please let me know. I'm more than welcome to refine the current configuration to the extent you expect.

Thanks in advance!