Summary

Do some simple refactoring of the post command.

Review Request #5118 — Created Dec. 16, 2013 and submitted Dec. 16, 2013, 4:51 p.m.

Information

Owner

david

Repository

RBTools

Branch

master

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description

Do some simple refactoring of the post command.

This change does some cleanup that struck me as I was reading through the post
implementation. I've moved the score matching out into a new utils file, and
rearranged/commented the argument evaluation a little bit.

Testing Done


Ran unit tests
Posted a change

Expand All
Collapse All

Ship it!

Looks good.

The only thing that stands out to me is that I have no idea how to use Score without going through the post code. It'd be nice if we could have the docstrings fleshed out a little more.

david Dec. 16, 2013, 4:51 p.m.
```
Sure. Done and pushed.
```

Status:: Completed
Change Summary:: Pushed to master (58ea479)