Skip to content

Rename http client package - #371

Merged
bastischubert merged 5 commits into
prometheus-community:mainfrom
Philldomd:feature/rename-http-client-pkg
Mar 10, 2026
Merged

Rename http client package#371
bastischubert merged 5 commits into
prometheus-community:mainfrom
Philldomd:feature/rename-http-client-pkg

Conversation

@Philldomd

@Philldomd Philldomd commented Mar 3, 2026

Copy link
Copy Markdown
Contributor

The old package http produced a conflict with net/http from std library

Also removed archived from readme

The old package http produced a conflict with net/http from std library

Signed-off-by: Örnfeldt Philip (66140321) <philip.ornfeldt@forsakringskassan.se>
Signed-off-by: Örnfeldt Philip (66140321) <philip.ornfeldt@forsakringskassan.se>
@Philldomd
Philldomd force-pushed the feature/rename-http-client-pkg branch from 20e9179 to e258739 Compare March 3, 2026 08:42
@Philldomd

Copy link
Copy Markdown
Contributor Author

The lint and CI build will not handle this change correctly since the dependecy does not exist yet.

Signed-off-by: Örnfeldt Philip (66140321) <philip.ornfeldt@forsakringskassan.se>
@bastischubert

Copy link
Copy Markdown
Contributor

@Philldomd we'd need to resolve the conflict on the bgp_neigbors (remember the rename of the function old -> 7_4?) before merging :)

Signed-off-by: Örnfeldt Philip (66140321) <philip.ornfeldt@forsakringskassan.se>
@Philldomd
Philldomd force-pushed the feature/rename-http-client-pkg branch from cba89cd to 10d7443 Compare March 6, 2026 10:41
@bastischubert
bastischubert merged commit 8d406ff into prometheus-community:main Mar 10, 2026
9 checks passed
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.

2 participants