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

Skip bcrypt in test env #190

Merged
merged 2 commits into from
Dec 5, 2023
Merged

Skip bcrypt in test env #190

merged 2 commits into from
Dec 5, 2023

Conversation

mikker
Copy link
Owner

@mikker mikker commented Dec 5, 2023

This could make everyone's test suites a bunch faster

@mikker mikker self-assigned this Dec 5, 2023
@mikker mikker merged commit 28edccd into master Dec 5, 2023
3 checks passed
@mikker mikker deleted the turn-off-bcrypt-in-test branch December 5, 2023 08:36
@mikker
Copy link
Owner Author

mikker commented Dec 5, 2023

Some anecdata from my own app:

$ time bin/rspec

# before
bin/rspec  59.04s user 4.86s system 88% cpu 1:12.59 total

# after
bin/rspec  28.33s user 4.54s system 74% cpu 44.261 total

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.

1 participant