Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Fix second TYPE line for metric name error when prometheus try to scrape. #7

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

Algorab
Copy link

@Algorab Algorab commented Apr 6, 2022

Description

Prometheus throw an error when the scraped metrics contains multiple identical type definitions.

Fixes

Fix second TYPE line for metric name error when prometheus try to scrape.

algorab added 3 commits April 6, 2022 12:19
…ape.

Prometheus throw an error when the scraped metrics contains multiple identical type definitions.
@Algorab
Copy link
Author

Algorab commented Apr 28, 2022

When run the same command as in the build pipeline for the formatting checks locally on the changes in the pull request I don't get an error:
➜ make lint
cd src; npm run lint

[email protected] lint
eslint . && prettier --check *.js

Checking formatting...
All matched files use Prettier code style!

Don't know why the check fail in the build pipeline, especially I didn't not change anything at this line.
Need help.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant