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

Check if recoverable is enabled in case allow_password_change is used #995

Merged

Conversation

MaicolBen
Copy link
Collaborator

Because projects that update the gem won't have the new field allow_password_change, this prevents asking if the recoverable module is added. Also, I fixed the broken url of running a single test.

@MaicolBen MaicolBen mentioned this pull request Oct 24, 2017
@zachfeldman zachfeldman self-requested a review October 24, 2017 17:59
@@ -28,4 +28,4 @@ bundle exec guard

Guard will re-run each test suite when changes are made to its corresponding files.

To run **just one test**: Flavio Castelli blogged about [how to execute a single unit test (or even a single test method)](https://webcache.googleusercontent.com/search?q=cache:lVNaE5lsPq0J:http://flavio.castelli.name/2010/05/28/rails_execute_single_test%2Bflavio.castelli.name/2010/05/28/rails_execute_single_test+!g&num=1&hl=en&strip=1&vwsrc=0) instead of running the complete unit test suite.
To run **just one test**: Flavio Castelli blogged about [how to execute a single unit test (or even a single test method)](https://flavio.castelli.me/2010/05/28/rails_execute_single_test/) instead of running the complete unit test suite.
Copy link
Contributor

Choose a reason for hiding this comment

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

You're right, this URL works...

@zachfeldman
Copy link
Contributor

Fine with me, waiting on one other approval to merge.

@lynndylanhurley
Copy link
Owner

Good catch, thanks @MaicolBen !

@MaicolBen MaicolBen merged commit fed0ed1 into lynndylanhurley:master Nov 14, 2017
@MaicolBen MaicolBen deleted the check_if_recoverable_is_enabled branch November 14, 2017 16:10
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