search

tower-archive / tower

UNMAINTAINED - Small components for building apps, manipulating data, and automating a distributed infrastructure.
http://tower.github.io
MIT License
1.79k stars 120 forks source link

Issue #365 - Added destroy command #372

Closed btbinhtran closed 11 years ago

btbinhtran commented 11 years ago

I added a destroy command, refactored generator code to work with Tower.root to make generate destroy commands testable within test/example app.

Color coded logger for create actions. Also bolded the logger actions.

Updated the ulimit in the Makefile to 1000, so it can work on OS X Lion.

Destroy command handles:

Need to look into handling namespaces like in Rails. Example:

tower generate model Animal::Bird would generate app/models/animal/bird.rb

and

tower destroy model Animal::Bird would delete app/models/animal/bird.rb and all the references.

thehydroimpulse commented 11 years ago

Could you concatenate all those commits into one?

btbinhtran commented 11 years ago

I concatenated all the commits into one which solves issue 365 and 371, but I'm not sure why the pull request closed on my end.

I had to revert my branch because I did some additional commits on top of master, then rebase.

When I pushed the rebased issue365destroycommand branch it closed the pull request.

Hope all is well.

thehydroimpulse commented 11 years ago

yeah that's weird. I heard Lance (viatropos) is fairly busy right now, so don't worry if your PR is sitting there for a little bit.

edubkendo commented 11 years ago

I sent him a message to take a look, so hopefully he'll get to this one soon.