API review

Proposer: Ken Conley

Present at review:

  • Gerkey

Question / concerns / comments

Review of rosunit is mostly covered by REP 100. rosunit is essentially a migration of roslaunch/rostest related tools used by rosbuild. The names of the executables have been cleaned up as part of this move.

The only true API change is that the test name is now optional, passed in by --name.

  • Brian:

    • The API of rosunit itself looks fine to me. It's happily being used by rosbuild.

    • rosbuild also uses several helper scripts in the rosunit package, and I was about to suggest that they be documented, when I found that they're already documented, on the rosunit page. Nice.

Meeting agenda

To be filled out by proposer based on comments gathered during API review period

Conclusion

Package status change mark change manifest)

  • /!\ Action items that need to be taken.

  • {X} Major issues that need to be resolved

Conclusion: API reviewed


Wiki: rosunit/Reviews/2011-01 Diamondback API Review_API_Review (last edited 2011-01-12 18:20:23 by KenConley)