Skip to content
This repository has been archived by the owner on Dec 5, 2019. It is now read-only.

Rake -> Thor #152

Merged
merged 1 commit into from
Nov 18, 2014
Merged

Rake -> Thor #152

merged 1 commit into from
Nov 18, 2014

Conversation

maxlinc
Copy link
Collaborator

@maxlinc maxlinc commented Nov 11, 2014

I prefer Thor (w/ or w/out a CLI) for a few reasons:

  • You can use it like Rake, but it's also better as CLIs. That's better for Pacto to become a polyglot project (Ruby bindings available, but CLI + server can be used for non-Ruby projects)
  • Much better handling of arguments and options so you can do things like "--strict", vs Rake's hacky way of passing parameters.
  • Better documentation for commands w/ descriptions of available options/arguments.
  • Easier to test - they're just Ruby classes.
  • A lot of useful helpers (see Thor::Shell and Thor::Actions especially)

@maxlinc maxlinc changed the title Start creating a Pacto CLI tool with Thor, and use that to replace the existing Rake tasks. Rake -> Thor Nov 11, 2014
@coveralls
Copy link

Coverage Status

Coverage increased (+1.2%) when pulling 6738e8e on maxlinc:thor into 89185cc on thoughtworks:master.

@maxlinc maxlinc force-pushed the thor branch 3 times, most recently from 50f50d9 to cffa32c Compare November 18, 2014 21:55
@coveralls
Copy link

Coverage Status

Coverage decreased (-0.04%) when pulling 50f50d9 on maxlinc:thor into 4498654 on thoughtworks:master.

@maxlinc maxlinc merged commit d5302e8 into thoughtworks:master Nov 18, 2014
@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) when pulling d5302e8 on maxlinc:thor into 4498654 on thoughtworks:master.

@maxlinc maxlinc deleted the thor branch November 24, 2014 20:35
@maxlinc maxlinc mentioned this pull request Dec 19, 2014
4 tasks
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants