Skip CI if file changes are .md or .rst

[davetcoleman]

This would save build time.

For inspiration, see #73

[rhaschke]

IMHO adding such extra rules doesn't make sense. We are using moveit_ci for both, C++ compiling (which is already sped up via ccache) and for moveit_tutorials / docs. The latter needs to run for .rst files as well.

[davetcoleman]

Interesting point, but I don't think this this will affect that.

• If a change to moveit_tutorials is only for .rst, there is no way i can see it breaking the code build, so we don't need moveit_ci
• The webpage testing of moveit_tutorials would still run, after moveit_ci is skipped. See here.

This would help with other use case of ours for moveit_ci.

[rhaschke]

My concern is mainly that such a change renders moveit_ci very specific to our MoveIt use case.
On the other hand, we can "only" save ~10min of building time.

[davetcoleman]

I don't think its specific... what project would require you to test all code when only a documentation file is updated? None.

10 min... I thought we were hitting time limits on Travis sometimes? Like 50 min.

Also regarding "very specific" to moveit, that is the name of the repo :-)

Anyway, I'm not asking you to add this feature, but rather our intern @simonGoldstein

[rhaschke]

If you insist, go ahead. I just wanted to formulate my concerns, which I have done.
Although Simon will implement the feature, I will be probably the one who needs to review and maintain this feature in future 😉