ros2 / rmw_iceoryx

rmw implementation for iceoryx
Apache License 2.0
144 stars 27 forks source link

Add basic ci #16

Closed jeising closed 4 years ago

jeising commented 4 years ago

Add CI for:

Resolves: #10

jeising commented 4 years ago

@michael-poehnl: There seems to be some inconsistency with uncrustify on current master. see: ci test results I fixed it in #12

Karsten1987 commented 4 years ago

Thanks @jeising. I think it's a great starting point. We could also consider adding the linters in a separate action. See here for examples: https://github.com/ros-controls/ros2_controllers/pull/33/files

As or the current uncrustify failures, they are going to be addressed in #15

jeising commented 4 years ago

I have no write access so @Karsten1987 feel free to merge it.

Karsten1987 commented 4 years ago

Looking at #15 it might be worth to limit the amount of jobs of push to

push:
    branches:
      - master
jeising commented 4 years ago

Lets put it there as soon as we need it :)