ignore modules option

[tsloughter]

Using https://github.com/tomas-abrahamsson/gpb there is generated code like:

e_type_string(S, Bin, _TrUserData) ->
    Utf8 = unicode:characters_to_binary(S),
    Bin2 = e_varint(byte_size(Utf8), Bin),
    <<Bin2/binary, Utf8/binary>>.

Since unicode:characters_to_binary/1 can return an error or incomplete tuple the byte_size function call fails to type check.

With gradualizer I ignore these modules:

{gradualizer_opts, [{exclude_modules, [opentelemetry_exporter_trace_service_pb,
                                       opentelemetry_exporter_metrics_service_pb,
                                       opentelemetry_exporter_logs_service_pb,
                                       opentelemetry_zipkin_pb]}]}.

It'd be nice if eqwalizer had a similar option.

[ilya-klyuchnikov]

As pointed out in WhatsApp/eqwalizer#37 (comment) - we have recently released ELP which has more options and flexibility for running eqwalizer. And it does support ignoring modules via putting -eqwalizer(ignore). in the module's body.

[tsloughter]

Thanks I noticed ELP this morning and am actually trying it out.

But adding to a module body doesn't help here since the module is generated. I'd just put the %% eqwalizer:ignore if it wasn't a generated module.

[ilya-klyuchnikov]

This issue belongs now more to ELP (in terms of managing options and configurations). Transferring to ELP