search

BCLab-UNM / SwarmBaseCode-ROS

The base code provided to teams participating in NASA Swarmathon IV.
http://nasaswarmathon.com
MIT License
46 stars 46 forks source link

Updated abridge and sbridge to Meet Style Guide Requirements #213

Closed kristiana-rendon closed 6 years ago

kristiana-rendon commented 6 years ago

Updated indentation spaces/tab to 2 spaces, variable names to COBOL case, function names to camel case, function/class/conditional statements have "{" and "}" on their own line, and other style guide requirements. Corrected typos. Changed variable names for consistency. Writing code that matches the style guide improves readability, comprehension, and overall code uniformity/aesthetic.

kristiana-rendon commented 6 years ago

Those have already been changed. Can you show me where you found that instance?

gmfricke commented 6 years ago

Sure. Remind me at the dev meeting tomorrow.

From: Kristiana Rendon notifications@github.com Sent: Tuesday, October 23, 2018 1:06 PM To: BCLab-UNM/SwarmBaseCode-ROS SwarmBaseCode-ROS@noreply.github.com Cc: Matthew Fricke matthew@fricke.co.uk; Review requested review_requested@noreply.github.com Subject: Re: [BCLab-UNM/SwarmBaseCode-ROS] Updated abridge and sbridge to Meet Style Guide Requirements (#213)

Those have already been changed. Can you show me where you found that instance?

— You are receiving this because your review was requested. Reply to this email directly, view it on GitHubhttps://github.com/BCLab-UNM/SwarmBaseCode-ROS/pull/213#issuecomment-432379067, or mute the threadhttps://github.com/notifications/unsubscribe-auth/ADw5E-Hb7BRIm3zy3MBKpldrTYDsVlgkks5un2ingaJpZM4X0XKb.