Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Issue #34 - Add rake task to remove existing seed files #48

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ltk
Copy link
Contributor

@ltk ltk commented Oct 31, 2014

Updated version of #44.

Brings the purge task into its own class for testability, and adds test coverage.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling a170852 on rake-purge into 59bc4de on master.

@ltk
Copy link
Contributor Author

ltk commented Oct 31, 2014

Need to add a blurb about this rake task to the readme.

def deletion_confirmed?
log_debug "Are you sure you want to delete these files? (Type 'yes' to confirm.)"
input = $stdin.gets.chomp
input == 'yes' ? true : false
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't need the ternary here. input == 'yes' will give you true or false.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

copy pasta. will update

@rysospicy
Copy link
Contributor

Update dat README and 💥 goes the dynamite.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants