crontab icon indicating copy to clipboard operation
crontab copied to clipboard

Warning on Elixir v1.14

Open thbar opened this issue 3 years ago • 4 comments

I'm doing a bit of preliminary testing with https://github.com/elixir-lang/elixir/releases/tag/v1.14.0-rc.1, and noticed the following warning, so I'm opening an issue to report it :

==> crontab
Compiling 7 files (.ex)
warning: Application.get_env/3 is discouraged in the module body, use Application.compile_env/3 instead
  lib/crontab/scheduler.ex:21: Crontab.Scheduler

Generated crontab app

Apparently not fixed on master:

https://github.com/jshmrtn/crontab/blob/c7641a424abc1191c3bf91ace369350097eb4890/lib/crontab/scheduler.ex#L21

thbar avatar Aug 17 '22 11:08 thbar

Fixed by 81bd15abaa954afc8e8f9d1fd5b295fb3bf59139

maennchen avatar Feb 02 '23 12:02 maennchen

Looks like the warning is still being triggered:

Compiling 7 files (.ex)
    warning: Application.get_env/3 is discouraged in the module body, use Application.compile_env/3 instead
    │
 21 │     @max_runs Application.get_env(:crontab, :max_runs, 10_000)
    │                           ~
    │
    └─ lib/crontab/scheduler.ex:21:27: Crontab.Scheduler (module)

sergchernata avatar Mar 28 '24 17:03 sergchernata

Same here! Maybe we can reopen a new issue @sergchernata?

thbar avatar Mar 28 '24 22:03 thbar

It seems this has been resolved in #116, but there has not been a new release?

Hermanverschooten avatar Jun 26 '24 08:06 Hermanverschooten