Upgrade plugins and fix new linter errors

Author: NelsonVides

rebar.config

@@ -74,11 +74,11 @@
 ]}.
 
 {project_plugins, [
-    {rebar3_hex, "~> 7.0.8"},
-    {rebar3_lint, "~> 3.2.6"},
-    {rebar3_ex_doc, "~> 0.2.25"},
-    {erlfmt, "~> 1.6.0"},
-    {covertool, "~> 2.0.7"}
+    {rebar3_hex, "~> 7.0"},
+    {rebar3_lint, "~> 4.1"},
+    {rebar3_ex_doc, "~> 0.2"},
+    {erlfmt, "~> 1.7"},
+    {covertool, "~> 2.0"}
 ]}.
 
 {hex, [{doc, #{provider => ex_doc}}]}.

src/collectors/vm/prometheus_vm_system_info_collector.erl

@@ -222,12 +222,12 @@ metric_enabled(Name, Metrics) ->
 
 collect_allocator_metrics() ->
     Metrics = lists:flatten([
-        collect_allocator_metrics_(Alloc, Instance, Info)
+        collect_allocator_metrics_1(Alloc, Instance, Info)
      || {{Alloc, Instance}, Info} <- allocators()
     ]),
     prometheus_model_helpers:gauge_metrics(Metrics).
 
-collect_allocator_metrics_(Alloc, Instance, Info) ->
+collect_allocator_metrics_1(Alloc, Instance, Info) ->
     [
         [
             allocator_metric(Alloc, Instance, Kind, Key, KindInfo)
@@ -255,8 +255,7 @@ allocators() ->
     [
         {{A, N}, lists:sort(proplists:delete(versions, Props))}
      || A <- Allocators,
-        Allocs <- [erlang:system_info({allocator, A})],
-        Allocs =/= false,
+        [_ | _] = Allocs <- [erlang:system_info({allocator, A})],
         {_, N, Props} <- Allocs
     ].
 

src/metrics/prometheus_boolean.erl

@@ -173,7 +173,7 @@ Raises:
     Value :: prometheus:prometheus_boolean().
 set(Registry, Name, LabelValues, Value) ->
     Value1 = prometheus_model_helpers:boolean_value(Value),
-    set_(Registry, Name, LabelValues, Value1).
+    set_1(Registry, Name, LabelValues, Value1).
 
 ?DOC(#{equiv => toggle(default, Name, [])}).
 -spec toggle(prometheus_metric:name()) -> ok.
@@ -348,12 +348,12 @@ collect_metrics(Name, {CLabels, Labels, Registry}) ->
 deregister_select(Registry, Name) ->
     [{{{Registry, Name, '_'}, '_'}, [], [true]}].
 
-set_(Registry, Name, LabelValues, Value) ->
+set_1(Registry, Name, LabelValues, Value) ->
     Key = {Registry, Name, LabelValues},
     Spec = {?BOOLEAN_POS, Value},
     case ets:update_element(?TABLE, Key, Spec) of
         false ->
-            insert_metric(Registry, Name, LabelValues, Value, fun set_/4);
+            insert_metric(Registry, Name, LabelValues, Value, fun set_1/4);
         true ->
             ok
     end.

test/eunit/metric/prometheus_quantile_summary_tests.erl

@@ -478,4 +478,6 @@ collect_monitors([{Pid, Ref} | Monitors]) ->
         {'DOWN', Ref, process, Pid, Reason} ->
             ?assertEqual(normal, Reason),
             collect_monitors(Monitors)
+    after 5000 ->
+        ct:fail("Tasks never finished in a reasonable amount of time")
     end.