| Commit message (Collapse) | Author | Age | Files | Lines |
| |
|
|
|
|
|
| |
We never used this.
Removed, as okayed by cwebber.
|
|
|
|
|
|
|
|
|
|
|
| |
1) destroy_self() is a horrible function name, make it "destroy".
workbench.destroy() is descriptive enough.
2) WorkbenchManager.create_workbench() -> WorkbenchManager.create()
We use the pattern "with workbench_manager.create() as workbench:"
No need to mention workbenches three times in a row...
Signed-off-by: Sebastian Spaeth <Sebastian@SSpaeth.de>
|
|
|
|
|
|
|
| |
This allows us to use "with Workbench() as foo: do_stuff..."
No consumers have been switched yet though.
Signed-off-by: Sebastian Spaeth <Sebastian@SSpaeth.de>
|
| |
|
|\
| |
| |
| |
| |
| | |
Conflicts:
mediagoblin/db/migrations.py
mediagoblin/submit/views.py
|
| | |
|
|/
|
|
|
|
|
| |
* Removed trailing whitespace
* Line length < 80 where possible
* Honor conventions on number of blank lines
* Honor conventions about spaces around :, =
|
| |
|
|
|
|
|
| |
And add a lot of warnings, as the checks for "being part of
the main Manager" are all gone.
|
|
|
|
|
|
|
|
| |
As Workbench has the localized_file() method, use this
everywhere and drop the wrapper method from
WorkbenchManager.
The processing code already did that.
|
|
|
|
|
| |
Some references to Workbench.dir look ugly, I'm happy to
hear suggestions there.
|
| |
|
|
|
|
| |
figure that out by looking to see whether our storage is local or not.
|
|
|