-
Notifications
You must be signed in to change notification settings - Fork 19
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
Final pre-release nits #89
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.
LGTM! I added few minor comments.
EDIT: the tangobot_app package doesn't have the maintainer set properly, and some comments can be deleted.
tangobot_app/app/build.gradle
Outdated
compile 'org.ros.rosjava_messages:std_msgs:[0.3,)' | ||
compile 'org.ros.rosjava_messages:sensor_msgs:[0.3,)' | ||
compile 'org.ros.rosjava_messages:geometry_msgs:[0.3,)' | ||
compile 'org.ros.rosjava_core:rosjava_helpers:[0.3, 0.4)' |
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.
Perhaps we could set the correct version for the messages, as they are quite different from the numbers of rosjava releases.
tangobot_app/app/build.gradle
Outdated
minSdkVersion 15 | ||
targetSdkVersion 22 | ||
versionName project.catkin.pkg.version | ||
versionCode 1 |
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 would add a small comment to remind us that the version code has to be updated manually along with the package's minor version number.
Good ideas, @jubeira , thank you. Addressed. |
This adds a few final changes prior to the initial release:
@jubeira could you please review? Also let me know if you have any other observations prior to release