[MESOS-1882] Add a --dry-run to verify-reviews.py.

• Type: Improvement
• Status: Open
• Priority: Minor
• Resolution: Unresolved
• Component/s: reviewbot
• Labels: newbie
• Assignee: Unassigned
• Reporter: Benjamin Mahler
• Shepherd: Benjamin Mahler
• Created: 09/Oct/14 01:48
• Updated: 29/Apr/19 09:26

Description

To improve the ease of making changes to verify-reviews.py, we should add the ability to pass a --dry-run flag. This will print all commands to be executed.

Additional improvements that we may want to break out of this ticket:

1. Rename verify-reviews.py to verify_reviews.py to allow importing.
2. Make verify-reviews.py only execute when run as a __main__, if imported it should merely make the library methods / classes available, so that one can use the library from an interpreter.