Skip to content

Remove unused scraper - #1790

Open
jeschkies wants to merge 1 commit into
prometheus-community:masterfrom
jeschkies:karsten/remove-unused-scraper
Open

Remove unused scraper#1790
jeschkies wants to merge 1 commit into
prometheus-community:masterfrom
jeschkies:karsten/remove-unused-scraper

Conversation

@jeschkies

Copy link
Copy Markdown
Contributor

This removes unused code.

Signed-off-by: jeschkies <karsten.jeschkies@grafana.com>
@jeschkies
jeschkies force-pushed the karsten/remove-unused-scraper branch from bcde478 to 9b08e8d Compare December 8, 2025 16:23
@andriikushch

Copy link
Copy Markdown
Contributor

Hi @jeschkies 👋

This code was included in the last release.

This PR will remove the mentioned methods from the library API; therefore, it will be a breaking change.

It is possible to check whether these methods/types are used in this project; however, some of them are exported so they can be used programmatically (as a library) by other projects.

At the same time, there is no major version release yet, so we can consider this change if we have a good reason to remove this functionality.

@cristiangreco, what is your opinion on that?

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.

3 participants