Incorrect conversion of Timex.now/1 to DateTime.utc_now/1

[kerryb]

Versions

• Elixir: 1.15.2 (compiled with Erlang/OTP 26)
• Styler: 0.8.3

Example Input

Timex.now("Europe/London")

Stacktrace / Current Behaviour

Styler converts Timex.now/1 to DateTime.utc_now/1, which expects a calendar or unit (not a timezone). This is the formatted code:

DateTime.utc_now("Europe/London")

I believe the correct conversion should be:

DateTime.now!("Europe/London")

[novaugust]

:O thanks for the bug report! will have that fixed in a jiffy

[novaugust]

fixed on main, will publish a release at the end of the workday =) thanks again for the report (and even moreso for telling me what it should be!)

[kerryb]

Thanks for the quick fix!

[novaugust]

released v0.8.4, hope this works better across the pond now :)

[sabiwara]

Hello! I noticed that this issue is still happening when using the pipe operator:

"Europe/London" |> Timex.now() |> dbg()
# becomes
"Europe/London" |> DateTime.utc_now() |> dbg()

The fix is happening on the node (apparent arity 0), but doesn't account for the effective arity of the call due to the pipe (arity 1).

[novaugust]

ahh thank you sabiwara, and apologies for missing that. will get a fix up

[novaugust]

published 0.9.5 with fix for the pipes problem. cheers

[sabiwara]

Thanks Matt for the fix and for the awesome project! Cheers 💜