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: update dependencies #72

Merged
merged 2 commits into from
Jan 10, 2024
Merged

fix: update dependencies #72

merged 2 commits into from
Jan 10, 2024

Conversation

ctrox
Copy link
Contributor

@ctrox ctrox commented Dec 22, 2023

this mainly upgrades github.com/ninech/apis and loki because of some broken dependencies. The loki upgrade required some changes to the fake client since the interface changed slightly. The other test changes are due to a behaviour changes of the k8s fake client.

@ctrox ctrox force-pushed the update-deps branch 2 times, most recently from 113f8c9 to 1c29a13 Compare December 22, 2023 15:47
api/client.go Show resolved Hide resolved
this mainly upgrades github.com/ninech/apis and loki because of some
broken dependencies. The loki upgrade required some changes to the fake
client since the interface changed slightly. The other test changes are
due to a behaviour changes of the k8s fake client.
@ctrox
Copy link
Contributor Author

ctrox commented Jan 10, 2024

@gajicdev FYI I added another commit to replace the deprecated k8s.io/utils/pointer package as the linter complained.

@ctrox ctrox merged commit ac850c0 into main Jan 10, 2024
2 checks passed
@ctrox ctrox deleted the update-deps branch January 10, 2024 14:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants