Commit ca53b57
committed
Stringify non-string job names in push client
We do this to all other label values in the client and we should make
this one consistent. Right now it's particularly surprising that passing
a symbol as the job name results in an error like:
```
undefined method `include?' for :foo:Symbol (NoMethodError)
```
Signed-off-by: Chris Sinjakli <chris@sinjakli.co.uk>1 parent e144d62 commit ca53b57
File tree
2 files changed
+10
-0
lines changed- lib/prometheus/client
- spec/prometheus/client
2 files changed
+10
-0
lines changed| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
87 | 87 | | |
88 | 88 | | |
89 | 89 | | |
| 90 | + | |
| 91 | + | |
90 | 92 | | |
91 | 93 | | |
92 | 94 | | |
| |||
| Original file line number | Diff line number | Diff line change | |
|---|---|---|---|
| |||
93 | 93 | | |
94 | 94 | | |
95 | 95 | | |
| 96 | + | |
| 97 | + | |
| 98 | + | |
| 99 | + | |
| 100 | + | |
| 101 | + | |
| 102 | + | |
| 103 | + | |
96 | 104 | | |
97 | 105 | | |
98 | 106 | | |
| |||
0 commit comments