-
Notifications
You must be signed in to change notification settings - Fork 20
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
distro: Add WithIDGenerator option #2634
Conversation
CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅ |
I have read the CLA Document and I hereby sign the CLA |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Can you also update the CHANGELOG.md
and fix issues reported by golangci-lint
?
Thank you for your comments @pellared . I already added your suggestions and the changelog |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #2634 +/- ##
==========================================
+ Coverage 78.43% 82.77% +4.33%
==========================================
Files 83 83
Lines 3223 3228 +5
==========================================
+ Hits 2528 2672 +144
+ Misses 617 483 -134
+ Partials 78 73 -5
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
do I have to create a signed commit? |
Yes. This may help you: https://superuser.com/questions/397149/can-you-gpg-sign-old-commits |
@sergiopastan We are just waiting for you to sign the commits. |
Bumps [github.com/goyek/x](https://github.com/goyek/x) from 0.1.5 to 0.1.6. - [Release notes](https://github.com/goyek/x/releases) - [Changelog](https://github.com/goyek/x/blob/main/CHANGELOG.md) - [Commits](goyek/x@v0.1.5...v0.1.6) --- updated-dependencies: - dependency-name: github.com/goyek/x dependency-type: direct:production update-type: version-update:semver-patch ... Signed-off-by: dependabot[bot] <support@github.com> Co-authored-by: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
* Update k8s.io/kube-openapi digest to 778a556 * go mod tidy --------- Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Robert Pająk <pellared@hotmail.com>
2c03943
to
14f9bdf
Compare
Hi @pellared I think is ok right now. Let me know if anything else is needed. Thanks! |
@sergiopastan All good. Thanks for your contribution 🎉 I will merge once the CI passes. |
This adds the WithIDGenerator option mentioned in this issue: #1377