Skip to content
Snippets Groups Projects
Unverified Commit c27ab9b8 authored by Dennis Schridde's avatar Dennis Schridde Committed by GitHub
Browse files

Do not load clojure.core/inc in metabase.analytics.prometheus (#32515)

Instead of silencing the linter, do not load the var.

Suggested-by: @camsaul
References: https://metaboat.slack.com/archives/CKZEMT1MJ/p1689785862424079
parent 65e02102
No related branches found
No related tags found
No related merge requests found
......@@ -5,6 +5,7 @@
Api is quite simple: [[setup!]] and [[shutdown!]]. After that you can retrieve metrics from
http://localhost:<prometheus-server-port>/metrics."
(:refer-clojure :exclude [inc])
(:require
[clojure.java.jmx :as jmx]
[iapetos.collector :as collector]
......@@ -251,7 +252,6 @@
(catch Exception e
(log/warn e (trs "Error stopping prometheus web-server"))))))))
#_{:clj-kondo/ignore [:redefined-var]}
(defn inc
"Call iapetos.core/inc on the metric in the global registry,
if it has already been initialized and the metric is registered."
......
0% Loading or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment