fastly / fastly-exporter

A Prometheus exporter for the Fastly Real-time Analytics API
Apache License 2.0
98 stars 36 forks source link

spelling on exclude #148

Closed justincely closed 8 months ago

justincely commented 1 year ago

Minor spelling correction.

peterbourgon commented 1 year ago

Elide isn't a misspelling of exclude, it was chosen deliberately 😇

mrnetops commented 1 year ago

While technically correct (and I love a new word!) given 154 million google hits for "exclude command line options" and ~3 orders of magnitude less with 416 thousand for "elide command line options", we might want to use the one closer to common parlance ;)

justincely commented 1 year ago

hah! @peterbourgon ty for educating me today ❤️

...Not gonna disagree with a move to common parlance though :)