Skip to content

internal/utils: Cleanup and IGL unification #310

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

Merged
merged 1 commit into from
Jul 15, 2025
Merged

Conversation

oxzi
Copy link
Member

@oxzi oxzi commented Apr 28, 2025

With https://github.com/Icinga/icinga-go-library/pull/64 and
https://github.com/Icinga/icinga-go-library/pull/127 being merged, most
of the internal/utils package is moved to the IGL and can be removed
here. This is another step to unify our Go codebase.

Draft until Icinga/icinga-go-library#127 got merged and IGL was bumped.

@cla-bot cla-bot bot added the cla/signed CLA is signed by all contributors of a PR label Apr 28, 2025
@oxzi oxzi force-pushed the drop-utils-use-igl branch 2 times, most recently from 5651b90 to 160c395 Compare April 30, 2025 13:16
@oxzi oxzi force-pushed the drop-utils-use-igl branch from 160c395 to 0167a08 Compare May 8, 2025 07:34
@oxzi oxzi changed the title Drop internal/utils package internal/utils: Cleanup and IGL unification May 8, 2025
With Icinga/icinga-go-library#64 and
Icinga/icinga-go-library#127 being merged, most
of the internal/utils package is moved to the IGL and can be removed
here. This is another step to unify our Go codebase.
@oxzi oxzi force-pushed the drop-utils-use-igl branch from 0167a08 to 3a6cd5e Compare June 30, 2025 12:36
@oxzi oxzi marked this pull request as ready for review June 30, 2025 12:36
@yhabteab yhabteab merged commit 266f62d into main Jul 15, 2025
26 checks passed
@yhabteab yhabteab deleted the drop-utils-use-igl branch July 15, 2025 14:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla/signed CLA is signed by all contributors of a PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants