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

Moves Timecop to development dependency, Issue #55 #61

Merged
merged 1 commit into from
May 28, 2017

Conversation

rdalin82
Copy link
Contributor

Resolves #55

Moves Timecop to development dependency
Use Timecop now_without_mock_time method when Timecop available. In
other case use the Time.raw_now method

…_time method when Timecop is available, but in other cases uses Time.raw_now method.
@ArturT ArturT merged commit 12642e1 into KnapsackPro:master May 28, 2017
@ArturT
Copy link
Member

ArturT commented May 28, 2017

@rdalin82 Thanks for PR. I've released a new knapsack version.

@rdalin82
Copy link
Contributor Author

@ArturT Thanks for maintaining an easy to contribute to project. Cheers!

@ArturT
Copy link
Member

ArturT commented May 29, 2017

@rdalin82 I'm curious, how you found out about it? Is it because of tag first-timers-only?

@rdalin82
Copy link
Contributor Author

@ArturT yes it was I did this as a search: label:first-timers-only language:ruby is:open no:assignee

@ArturT
Copy link
Member

ArturT commented May 29, 2017

Good to know that first-timers-only tag works :)

@rdalin82
Copy link
Contributor Author

It does! lots of articles are starting to reference it, I'm thinking about doing my own medium article about my experience.

@ArturT
Copy link
Member

ArturT commented May 29, 2017

Oh, that's cool. Please let me know if you publish something. Do you have a twitter account so I could follow you?

@rdalin82
Copy link
Contributor Author

I'm not on twitter at the moment, but I'll let you know if that changes in the future. I'll send you a message if I do publish.

@ArturT
Copy link
Member

ArturT commented May 29, 2017

Awesome :)

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.

2 participants