RBTools 0.7 introduced support for aliases, a handy way to simplify common tasks and workflows when working with source code and review requests.
With aliases, you can write your own RBTools commands that expand into, well, anything. They could be a set of common options for an existing command, or they could call out to shell commands.
You can define your own aliases for your personal use, but you can also define aliases for your entire team. Since they live in .reviewboardrc, aliases can be committed to your repository for everyone to use. This is quite powerful, as it can help to easily standardize workflows for your whole team.
I’m going to show you a few ways to compose workflows using aliases.
Simplify posting code for review
Feature branches in Git are great. They help you to stay organized when you have lots of changes that depend on each other. If you’ve made heavy use of them, you’ve probably used this workflow a bit:
# Post some code...
$ rbt post HEAD
# After making some changes to code...
$ rbt post -u HEAD
# After making some changes to the commit message...
$ rbt post -g yes -u HEAD
There aren’t a lot of parameters there, but you still have to remember them and type them every time. The -u HEAD or the -g yes -u HEAD gets repetitive.
Let’s add a few nice and short aliases in ~/.reviewboardrc:
ALIASES = {
# Post this
'pt': 'post HEAD',
# Post this and update
'ptu': 'post -u HEAD',
# Post this, update, and re-guess the description from the commit message
'ptug': 'post -g yes -u HEAD',
}
Now the workflow becomes simply:
# Post some code...
$ rbt pt
# After making some changes to code...
$ rbt ptu
# After making some changes to the commit message...
$ rbt ptug
Much shorter, and hopefully easier to remember.
Run unit tests before posting code
If your project has a unit test suite, you’re probably supposed to run it before posting code for review. It’s easy to forget, or to just ignore it, since it’s another step in your process. We can solve both of those problems.
Say your project has a ./scripts/run-tests.sh that runs your test suite and outputs 1 for failure, 0 for success. Let’s make an alias that ensures tests pass before posting, and we’ll even pre-populate the Testing Done field with a message stating they pass. Let’s call this p for short.
ALIASES = {
'p': '!./scripts/run-tests.sh &&'
'rbt post --testing-done="Unit tests pass." $*',
}
Now you’ll never forget to run tests, and you’ll even have less to type.
ProTip: Define this as an alias in the .reviewboardrc in your repository, and switch your team to use that instead of rbt post.
Sanity-check patches on a review request
If you’re the gatekeeper for your project’s code (perhaps you’re an open source developer?), you need to test the patches that other people write before landing them. Thanks to rbt patch, you can easily pull down a diff from a review request and apply it, but you still need to run your tests before landing it. You also need to be careful not to disturb any changes in your own tree, or to test on the wrong branch.
To really sandbox your testing, your ideal workflow may look something like this:
$ git clone -b master . /tmp/test-project-<review request id>
$ cd /tmp/test-project-<review request id>
$ rbt patch <review request id>
$ ./scripts/runtests.sh
$ cd -
$ rm -rf /tmp/test-project-<reviw request id>
Let’s make an alias for this:
ALIASES = {
'test-patch': '!rm -rf /tmp/test-project-$1 &&'
'git clone -b master . /tmp/test-project-$1 &&'
'cd /tmp/test-project-$1 &&'
'rbt patch $1 &&'
'./scripts/runtests.sh &&'
'rm -rf /tmp/test-project-$1',
}
Now you have a quick and easy way to check any patch applying to the repository you’re currently in, without messing up your repository, typing a bunch of commands, or worrying about directories.
Screenshot a window and attach it in one go
If you’re doing UI work, you’re posting screenshots a lot. This means taking a screenshot, saving it somewhere, and drag-and-dropping it into a review request with a caption.
Let’s get that down to one command.
Now, different operating systems have different tools for capturing a screenshot. I’ll show an example for MacOS X and another for Linux. For Windows, you’ll need to grab a command line screenshot tool.
ALIASES = {
# For MacOS X:
'attach-screenshot': '!screencapture -wo /tmp/screenshot.png &&'
'rbt attach --caption "$2" $1 /tmp/screenshot.png &&'
'rm /tmp/screenshot.png',
# For Linux:
'attach-screenshot': '!import /tmp/screenshot.png &&'
'rbt attach $* tmp/screenshot.png &&'
'rm /tmp/screenshot.png',
}
Now uploading a screenshot is quick and easy! To upload to review request #123, just run:
$ rbt attach-screenshot 123
You can even attach a caption or custom filename, like so:
$ rbt attach-screenshot --caption="About dialog" --filename=about-dlg.png 123
Share your workflows!
We’d love to hear about any workflows you come up with. Send them along. We may even feature them in a future post or as samples in the documentation!
