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

Feat/add comp rpt #39

Merged
merged 16 commits into from
Oct 4, 2018
Merged

Feat/add comp rpt #39

merged 16 commits into from
Oct 4, 2018

Conversation

JWhitleyWork
Copy link

No description provided.

@mlemm99
Copy link
Contributor

mlemm99 commented Oct 2, 2018

line 445 and 446 of DBC is missing a 1+ or 0+

@zoakesAStuff
Copy link
Contributor

Why is all this CI stuff in here too?

@JWhitleyWork
Copy link
Author

@zoakesAStuff - In order to use the new messages in astuff_sensor_msgs, I have to manually clone the astuff_sensor_msgs repo. I'm doing it with a shallow, sparse clone so it only gets the pacmod_msgs package. This will be removed in the future once the new astuff_sensor_msgs has been released to the official ROS repos.

@JWhitleyWork
Copy link
Author

@zoakesAStuff / @mlemm99 - The issues you brought up have been addressed and I added DBC validation to the CI stack. This will make sure that any future changes we make are, at the very least, automatically validated. Thanks.

Copy link
Contributor

@zoakesAStuff zoakesAStuff left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good

@JWhitleyWork JWhitleyWork merged commit dbdc3cc into master Oct 4, 2018
@JWhitleyWork JWhitleyWork deleted the feat/add_comp_rpt branch October 4, 2018 15:34
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