Merge lp:~statik/mup/stop-the-line into lp:~niemeyer/mup/trunk
Proposed by
Elliot Murphy
Status: | Rejected |
---|---|
Rejected by: | Gustavo Niemeyer |
Proposed branch: | lp:~statik/mup/stop-the-line |
Merge into: | lp:~niemeyer/mup/trunk |
To merge this branch: | bzr merge lp:~statik/mup/stop-the-line |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Gustavo Niemeyer | pre-implementation advice | Needs Fixing | |
Review via email: mp+2441@code.launchpad.net |
To post a comment you must log in.
Unmerged revisions
- 33. By Elliot Murphy
-
Merge trunk.
- 32. By Elliot Murphy
-
Added a couple of other unit tests.
- 31. By Elliot Murphy
-
fix some unused variable warnings
- 30. By Elliot Murphy
-
a basic skeleton of my plugin, with a failing test
I'm requesting a review of this branch as I'd like some implementation advice - it's not yet ready to land.
I think I've got the plugin skeleton hooked up correctly, and a basic entry point for unit tests of the plugin, but I'm not sure how to write a unit tests that checks to see if the plugin responds correctly to a command. A quick glance over the plugin and a tip or two on getting the unit test written will help me a lot, and should give me enough to figure out the rest of it. Thanks!