agrc / forklift

:tractor::package::sparkles: Slinging data all over the place :tractor::package::sparkles:
MIT License
28 stars 3 forks source link

Implement special-delivery command #286

Closed stdavis closed 5 years ago

stdavis commented 5 years ago

Description of Changes

This pull request adds a new special-delivery command.

Test results and coverage

image

Looks like the new options for lift and ship are in docopt but not implemented. Do we want to take the time to implement or revert those changes and add them later?

steveoh commented 5 years ago

I think we should create separate pr's for the new options.

Rebase this with master and remove the WIP commits as well and then I think this can be merged.

Great job with the tests and typos

stdavis commented 5 years ago

Done. I also added a [not yet implemented] note to the two new parameters just to make sure that there's no confusion.