Merge lp://staging/~ursinha/ubuntu-ci-services-itself/fix-paths-changesprocessor into lp://staging/ubuntu-ci-services-itself

Proposed by Ursula Junque
Status: Merged
Approved by: Chris Johnston
Approved revision: 79
Merged at revision: 83
Proposed branch: lp://staging/~ursinha/ubuntu-ci-services-itself/fix-paths-changesprocessor
Merge into: lp://staging/ubuntu-ci-services-itself
Diff against target: 150 lines (+54/-27)
2 files modified
cli/cli/changes_processor.py (+22/-14)
cli/tests/test_changes_processor.py (+32/-13)
To merge this branch: bzr merge lp://staging/~ursinha/ubuntu-ci-services-itself/fix-paths-changesprocessor
Reviewer Review Type Date Requested Status
Chris Johnston (community) Approve
Review via email: mp+200569@code.staging.launchpad.net

Commit message

Fixing the way ChangesProcessor handles paths, so we don't have to worry about setting the directory where the files to upload are; also added a support for a provided directory, if that's different than where the changes file is.

Description of the change

Fixing the way ChangesProcessor handles paths, so we don't have to worry about setting the directory where the files to upload are; also added a support for a provided directory, if that's different than where the changes file is.

To post a comment you must log in.
Revision history for this message
Chris Johnston (cjohnston) wrote :

 review approve
 merge approved

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches