Skip to content

Conversation

vimscientist69
Copy link

Fixes the following warning when using this package as a dependency in another swift project, specifically when running swift build

warning: 'swift-prometheus': found 1 file(s) which are unhandled; explicitly declare them as resources or exclude from the target

Fixes the following warning when using this package as a dependency in another swift project, specifically when running `swift build`

warning: 'swift-prometheus': found 1 file(s) which are unhandled; explicitly declare them as resources or exclude from the target
@incertum incertum added the 🔨 semver/patch No public API change. label Sep 26, 2025
Copy link
Contributor

@incertum incertum left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, and yes let's address the warning message.

Run

curl -s --retry 3 https://raw.githubusercontent.com/swiftlang/github-workflows/refs/heads/main/.github/workflows/scripts/check-swift-format.sh | bash

locally and then re-push to fix the formatting failure. (I'll improve the contributing docs asap.)

The other failure seems a perf improvement on current nightly-main, we'll check.

=== BEGIN DIFF ===
diff --git a/Benchmarks/Thresholds/nightly-main/PrometheusBenchmarks.DurationHistogram.p90.json b/Benchmarks/Thresholds/nightly-main/PrometheusBenchmarks.DurationHistogram.p90.json
index 282dc64..557fc88 100644
--- a/Benchmarks/Thresholds/nightly-main/PrometheusBenchmarks.DurationHistogram.p90.json
+++ b/Benchmarks/Thresholds/nightly-main/PrometheusBenchmarks.DurationHistogram.p90.json
@@ -1,3 +1,3 @@
 {
-  "mallocCountTotal" : 2
+  "mallocCountTotal" : 1
 }

@ktoso
Copy link
Collaborator

ktoso commented Oct 3, 2025

Looks good to me when formatting fixed, thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🔨 semver/patch No public API change.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants