AptFacade.perform_changes is unwieldy.
Bug #985493 reported by
Geoff Teale
This bug affects 1 person
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
Landscape Client |
Fix Released
|
Low
|
Geoff Teale |
Bug Description
When I was adding the package holds change activities into AptFacade on the client I noticed that the perform_changes method was becoming very unwieldy and hard to read. This could be, at least, broken down into multiple methods.
Related branches
lp:~tealeg/landscape-client/tidy-apt-facade-perform-changes
- Chris Glass (community): Approve
- Alberto Donato (community): Approve
-
Diff: 322 lines (+181/-102)2 files modifiedlandscape/package/facade.py (+167/-102)
landscape/package/tests/test_facade.py (+14/-0)
Changed in landscape-client: | |
status: | In Progress → Fix Committed |
tags: |
added: verification-done removed: verification-needed |
Changed in landscape-client: | |
status: | Fix Committed → Fix Released |
To post a comment you must log in.
qa + 1