Summary

Refactor rbt patch messages

Review Request #7300 — Created May 13, 2015 and discarded May 14, 2015, 8:51 p.m.

Information

Owner

brennie

Repository

RBTools

Branch

master

Bugs

Depends On

Reviewers

Groups

rbtools

People

Description

The messages in rbt patch have been refactored so that there is less
reptition in the code. We now keep string to use in our print
statements that say if we are patching or reverting.

Testing Done

Expand All
Collapse All

Tool: PEP8 Style Checker
Processed Files:
    rbtools/commands/patch.py



Tool: Pyflakes
Processed Files:
    rbtools/commands/patch.py

I think we're going to want to localize text before RBTools 1.0, and for localization, we can't use the trick of building a string with one or two words being provided as format arguments. While what we had was repetitive, it's also necessary for localization, so we'd end up having to revert all this down the road anyway.

Status:: Discarded