elixir-cldr / cldr

Elixir implementation of CLDR/ICU
Other
440 stars 33 forks source link

Documentation fixes #177

Closed petrus-jvrensburg closed 2 years ago

petrus-jvrensburg commented 2 years ago

I ran into a compilation error while setting up CLDR for a new project, so I updated the README. It was caused by the docstring being defined outside of a module:

== Compilation error in file lib/janus_web/cldr.ex ==
** (ArgumentError) cannot invoke @/1 outside module
    (elixir 1.13.3) lib/kernel.ex:6111: Kernel.assert_module_scope/3
    (elixir 1.13.3) expanding macro: Kernel.@/1
    lib/janus_web/cldr.ex:1: (file)
kipcole9 commented 2 years ago

@petrus-jvrensburg My sincere apologies, I have no idea which I wasn't notified of this PR. I appreciate the contribution and the thoughtfulness. Will publish a new docs version right away.

petrus-jvrensburg commented 2 years ago

Thanks @kipcole9 👍