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

Bump dependencies in eventing-manager #367

Conversation

friedrichwilken
Copy link
Contributor

@friedrichwilken friedrichwilken commented Dec 27, 2023

Description

To increase performance, stability and security this PR bumps the following dependencies

  • github.com/go-logr/logr
  • github.com/kyma-project/api-gateway
  • github.com/kyma-project/kyma/common/logging
  • github.com/vektra/mockery/v2

The changes in the API of api-gateway required some further code changes, because Methods are no longer of type string for validation.

Also, regenerated all generatet files due to the bump of mockery.

Bump github.com/go-logr/logr from v1.3.0 to v1.4.1.
bump github.com/kyma-project/kyma/common/logging
Bump github.com/vektra/mockery/v2 from v2.38.0 to v2.39.1.
Bump github.com/kyma-project/api-gatway. Due to API changes also adopt the usage of the module.
@friedrichwilken friedrichwilken requested a review from a team as a code owner December 27, 2023 09:37
@kyma-bot kyma-bot added cla: yes Indicates the PR's author has signed the CLA. size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Dec 27, 2023
@friedrichwilken friedrichwilken changed the title Bump dependencies in eventing manager Bump dependencies in eventing-manager Dec 27, 2023
pkg/object/apirule.go Outdated Show resolved Hide resolved
@kyma-bot kyma-bot added the lgtm Looks good to me! label Dec 27, 2023
@kyma-bot kyma-bot merged commit b9e1080 into kyma-project:main Dec 27, 2023
8 checks passed
@friedrichwilken friedrichwilken deleted the bump-dependencies-in-eventing-manager branch December 27, 2023 14:54
@friedrichwilken friedrichwilken self-assigned this Dec 28, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla: yes Indicates the PR's author has signed the CLA. lgtm Looks good to me! size/M Denotes a PR that changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants