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 controller-gen version #429

Merged
merged 8 commits into from
May 22, 2024

Conversation

helayoty
Copy link
Collaborator

Reason for Change:

  • Bumb controller-gen version to 0.15.0
  • Add make generate step in both unit tests and e2e tests pipelines.

Requirements

  • added unit tests and e2e tests (if applicable).

Issue Fixed:

Notes for Reviewers:

Signed-off-by: Heba Elayoty <hebaelayoty@gmail.com>
@codecov-commenter
Copy link

codecov-commenter commented May 21, 2024

Codecov Report

Attention: Patch coverage is 36.20690% with 74 lines in your changes are missing coverage. Please review.

Project coverage is 60.00%. Comparing base (b65dd2e) to head (cd9b9db).

Files Patch % Lines
api/v1alpha1/zz_generated.deepcopy.go 0.00% 52 Missing ⚠️
api/v1alpha1/params_validation.go 64.28% 10 Missing and 10 partials ⚠️
pkg/utils/common.go 75.00% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #429      +/-   ##
==========================================
- Coverage   61.09%   60.00%   -1.09%     
==========================================
  Files          29       29              
  Lines        2303     2403     +100     
==========================================
+ Hits         1407     1442      +35     
- Misses        828      887      +59     
- Partials       68       74       +6     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@helayoty helayoty enabled auto-merge (squash) May 22, 2024 00:20
@ishaansehgal99 ishaansehgal99 disabled auto-merge May 22, 2024 00:29
@ishaansehgal99 ishaansehgal99 merged commit 0dca449 into main May 22, 2024
6 checks passed
@ishaansehgal99 ishaansehgal99 deleted the helayoty/update-controller-gen branch May 22, 2024 00:29
smritidahal653 pushed a commit to smritidahal653/kaito that referenced this pull request Jun 7, 2024
**Reason for Change**:
- Bumb controller-gen version to 0.15.0
- Add make generate step in both unit tests and e2e tests pipelines.

**Requirements**

- [ ] added unit tests and e2e tests (if applicable).

**Issue Fixed**:
<!-- If this PR fixes GitHub issue 4321, add "Fixes #4321" to the next
line. -->

**Notes for Reviewers**:

---------

Signed-off-by: Heba Elayoty <hebaelayoty@gmail.com>
Co-authored-by: ishaansehgal99 <ishaanforthewin@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants