Let me first of all state, that I fully support this warning type. I find it good, that Elixir tries to guide developers in a generally right direction, but with the JSON-LD.ex <https://github.com/marcelotto/jsonld-ex> project, I find myself in a situation where I think it's not.
JSON-LD.ex is an implementation of the JSON-LD W3C standard, particularly its processing algorithms <https://www.w3.org/TR/json-ld-api/>. For this type of library, IMO maintainability is measured against the ability to conform the wording of the spec. The processing algorithms are quite complex and unfortunately specified in a very imperative style with many assignments in deeply nested conditionals. In the beginning, I fixed the warnings but found that this reduces the maintainability: I want to look at a numbered chunk of code and be able to easily compare it to the specs, to check if the implementation is correct or to adapt to changes in the spec (like the upcoming JSON-LD 1.1). So I discarded the fixes and started to ignore these warnings. Unfortunately, I'm now flooded with these warnings, which makes the whole warning system actually useless for this project. I've noticed this issue <https://github.com/elixir-lang/elixir/issues/5461> where this warning was already questioned, but at the same time making warnings optional ruled out in general. So, I'd like to know the current status and how I should proceed. I've already got a pull request <https://github.com/marcelotto/jsonld-ex/pull/3> that fixes the warnings, but I hesitate to merge it for above reasons. -- You received this message because you are subscribed to the Google Groups "elixir-lang-core" group. To unsubscribe from this group and stop receiving emails from it, send an email to [email protected]. To view this discussion on the web visit https://groups.google.com/d/msgid/elixir-lang-core/cfa222ff-64b8-4234-9f6b-c0035ec8ee27%40googlegroups.com. For more options, visit https://groups.google.com/d/optout.
