Update `CLDR` references to `Cldr` in docs

elixir-cldr / cldr_dates_times

Date & times formatting functions for the Common Locale Data Repository (CLDR) package https://github.com/elixir-cldr/cldr

Other

69 stars 13 forks source link

Update `CLDR` references to `Cldr` in docs #34

Closed sax closed 2 years ago

sax commented 2 years ago

When compiling docs in my applilcation, I see the following message every time I run mix docs:

11:29:35.752 [error] beam/beam_load.c(180): Error loading module 'Elixir.CLDR':
  module name in object code is 'Elixir.Cldr'

After applying the included changes to cldr_dates_times and cleaning my build directories, the error disappears.

kipcole9 commented 2 years ago

Thanks very much for the report. This has been bugging me too so good to get this resolved. I used your basic approach but kept the name "CLDR" since I use that to refer to the CLDR project as apposed to "Cldr" to refer to this implementation.

I've published ex_cldr_dates_times version 2.13.1 with the following changelog entry:

Bug Fixes

Fixes doc generation errors (no functional changes to executing code). Closes #34. Thanks to @sax for the report and the PR.

sax commented 2 years ago

Awesome, your version makes sense. Thanks for the release—I just verified that it fixes our errors.