Added :default param for I18n.localize

[fengb]

When using I18n.l, there are guards needed when a date is potentially nil:

date && I18n.l(date)
date ? I18n.l(date) : 'present'

This change will return the default when passed in:

I18n.l(date, default: nil)
I18n.l(date, default: 'present')

[radar]

Looks good to me, and a sensible alternative to what was discussed in #294.

I've applied this patch to a branch called pr-251, just to see if the Travis tests will pass there. If they do, then I will merge this to master.

Thanks for your contribution!

[kiramclean]

I notice a 2 year gap between this code being written and committed.. what's the timeline like for the next release that includes these changes?

[radar]

Middle/end of January is when I plan on doing the 0.8.0 release. Once things are less crazy

…

On 8 Dec. 2016, at 14:07, Kira McLean ***@***.***> wrote: I notice a 2 year gap between this code being written and committed.. what's the timeline like for the next release? — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or mute the thread.

[kiramclean]

Awesome. Is there a way to know what issues are blocking release? I'll have some free time over the next few weeks and would be happy to help.

[radar]

Take a look at the 0.8.0 milestone on GitHub. If you've got time to help; that would be appreciated. I'm happy to release 0.8.0 straight off master at the moment, but doing a release this close to the end of the year and when that release effects so many Rails versions... probably not a great idea.

…

On 9 Dec. 2016, at 06:48, Kira McLean ***@***.***> wrote: Awesome. Is there a way to know what issues are blocking release? I'll have some free time over the next few weeks and would be happy to help. — You are receiving this because you modified the open/close state. Reply to this email directly, view it on GitHub, or mute the thread.