-
Notifications
You must be signed in to change notification settings - Fork 78
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
Take .clang-format from moveit #81
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I read through the changes to .clang-format
(and compared it to the original file linked in the issue), and I think this looks OK!
e: Other than the build/test CI steps, though I don't think that's limited to just this issue.
It looks like something is broken in the cache. You'll need to go into the settings for this repo and delete the cache and run this again. |
So far, I haven't seen any possibility to delete the cache manually. Have you? |
I thought there was a way to do this in the repo settings. However it looks like this is not actually a feature and I'm probably just thinking of doing this in GitLab: actions/cache#2 A workaround for that would be to add an arbitrary version to the cache prefix like However looking more closely at this it seems that maybe there is something wrong with the melodic-source docker image that's causing this. |
It seems the |
f21f76c
to
edb0c9b
Compare
Codecov Report
@@ Coverage Diff @@
## master #81 +/- ##
=======================================
Coverage 72.75% 72.75%
=======================================
Files 10 10
Lines 554 554
=======================================
Hits 403 403
Misses 151 151 Continue to review full report at Codecov.
|
Not that different, in the end. Closes #72.