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

Railcop for avoiding Time.now #1405

Closed
dankohn opened this issue Oct 23, 2014 · 6 comments
Closed

Railcop for avoiding Time.now #1405

dankohn opened this issue Oct 23, 2014 · 6 comments

Comments

@dankohn
Copy link

dankohn commented Oct 23, 2014

Would you be interested in a Rubocop for Rails that autocorrects usage of Time.now to Time.zone.now?

The good and bad examples here could be searched for:
http://danilenko.org/2012/7/6/rails_timezones/

@bbatsov
Copy link
Collaborator

bbatsov commented Oct 26, 2014

Sounds reasonable. Btw, there's also Time.current.

@bbatsov
Copy link
Collaborator

bbatsov commented Jan 25, 2015

@dankohn Ping

@dankohn
Copy link
Author

dankohn commented Jan 25, 2015

Haven't been able to work on this. Please feel free to close and I'll
reopen when I have a pull request to look at.

Dan Kohn mailto:[email protected]
tel:+1-415-233-1000

On Sun, Jan 25, 2015 at 3:14 AM, Bozhidar Batsov [email protected]
wrote:

@dankohn https://github.com/dankohn Ping


Reply to this email directly or view it on GitHub
#1405 (comment).

@palkan
Copy link
Contributor

palkan commented Feb 19, 2015

@bbatsov @dankohn
I would like to work on this cop if you don't mind

@bbatsov
Copy link
Collaborator

bbatsov commented Feb 19, 2015

Fine by me.

On 19 February 2015 at 23:32, Vladimir Dementyev [email protected]
wrote:

@bbatsov https://github.com/bbatsov @dankohn
https://github.com/dankohn
I would like to work on this cop if you don't mind


Reply to this email directly or view it on GitHub
#1405 (comment).

Best Regards,
Bozhidar Batsov

http://www.batsov.com

@dankohn
Copy link
Author

dankohn commented Feb 19, 2015

I'd be thrilled. I haven't had a chance to jump into it.

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

No branches or pull requests

3 participants