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

chore(deps): bump dependencies and resolve conflicts #266

Merged
merged 4 commits into from
Sep 13, 2024
Merged

Conversation

floreks
Copy link
Member

@floreks floreks commented Sep 13, 2024

  • Bumped all direct dependencies
  • Resolved conflicts
  • Replaced the deprecated RateLimitingInterface with a new typed version TypedRateLimitingInterface[string]. It allows us to get rid of string cast check as queue items are typed now.
  • Regenerated manifests and added missing CRDs to helm chart

Docker Dependency

It looks like both OPA and helm dependencies import docker internally. Due to this we cannot get rid of it as a transitive dependency.

$ go mod graph | grep docker/docker
github.com/pluralsh/deployment-operator github.com/docker/[email protected]+incompatible
github.com/pluralsh/deployment-operator github.com/docker/[email protected]
github.com/docker/[email protected] github.com/danieljoos/[email protected]
github.com/docker/[email protected] golang.org/x/[email protected]
github.com/open-policy-agent/gatekeeper/[email protected] github.com/docker/[email protected]+incompatible
github.com/open-policy-agent/gatekeeper/[email protected] github.com/docker/[email protected]
helm.sh/helm/[email protected] github.com/docker/[email protected]+incompatible
helm.sh/helm/[email protected] github.com/docker/[email protected]
oras.land/[email protected] github.com/docker/[email protected]+incompatible
oras.land/[email protected] github.com/docker/[email protected]

@floreks floreks self-assigned this Sep 13, 2024
@floreks floreks merged commit 990aa13 into main Sep 13, 2024
32 of 33 checks passed
@floreks floreks deleted the bump/docker branch September 13, 2024 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants