Upgrade plugins and fix new linter errors

[NelsonVides]

@paulo-ferraz-oliveira tagging you as I know you were working on the last release of Elvis :)

[codecov]

Codecov Report

Attention: Patch coverage is 80.00000% with 2 lines in your changes missing coverage. Please review.

Files with missing lines	Patch %	Lines
...lectors/vm/prometheus_vm_system_info_collector.erl	85.71%	1 Missing ⚠️
...eunit/metric/prometheus_quantile_summary_tests.erl	0.00%	1 Missing ⚠️

Files with missing lines	Coverage Δ
src/metrics/prometheus_boolean.erl	93.84% <100.00%> (ø)
...lectors/vm/prometheus_vm_system_info_collector.erl	86.53% <85.71%> (-1.93%)	⬇️
...eunit/metric/prometheus_quantile_summary_tests.erl	98.05% <0.00%> (-0.48%)	⬇️

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[paulo-ferraz-oliveira]

Regarding

    [
        {{A, N}, lists:sort(proplists:delete(versions, Props))}
     || A <- Allocators,
        Allocs <- [erlang:system_info({allocator, A})],
        Allocs =/= false,
        {_, N, Props} <- Allocs
    ].

I haven't tested this, but

    [
        {{A, N}, lists:sort(proplists:delete(versions, Props))}
     || A <- Allocators,
        {_, N, Props} <- [erlang:system_info({allocator, A})]
    ].

should work, even if it's less readable (because you lose the filter, which is implicit).

[paulo-ferraz-oliveira]

Avoiding rules is also a possibility, but I tend not to do it at the project level, but start at the function level. Instead of disable you can always use ignore => [{M, F, A}].

[NelsonVides]

Ok, went around what the lists are supposed to be and found a different fix, it should be good to go now. Review plis? 🙏🏽😄

[paulo-ferraz-oliveira]

Hurray. That's what the rule is there for, to make us think about alternatives. If there's none we avoid it, but in this case there was a good solution for us 😄

[NelsonVides]

Actually, I just changed the code to make it more explicit, I find this list comprehension quite a terse hack, and also, what if SMP is disabled for ERTS (why tf would do that but you never know), then the number of allocators would be actually just one and this would return empty. Better make it just explicit, wdyt? :)

[paulo-ferraz-oliveira]

Oh, you changed it. Lemme give it a quick look.

[paulo-ferraz-oliveira]

I like explicit, yeah; if it works, it works. I just though you were going for minimal change, so the previous one (even if implicit) also worked for me.