Skip to content
This repository has been archived by the owner on Aug 12, 2024. It is now read-only.

Add tests to make sure rukpak enforces limitations #922

Merged
merged 1 commit into from
Jun 17, 2024

Conversation

m1kola
Copy link
Member

@m1kola m1kola commented Jun 17, 2024

Currently we do not allow:

  • Bundles with webhook definitions
  • Bundles which provide API Service definitions

Currently we do not allow:
* Bundles with webhook definitions
* Bundles which provide API Service definitions

Signed-off-by: Mikalai Radchuk <mradchuk@redhat.com>
Copy link

codecov bot commented Jun 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 14.60%. Comparing base (ad595f1) to head (2e6ca2e).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #922      +/-   ##
==========================================
+ Coverage   14.44%   14.60%   +0.15%     
==========================================
  Files          41       41              
  Lines        2554     2554              
==========================================
+ Hits          369      373       +4     
+ Misses       2128     2126       -2     
+ Partials       57       55       -2     

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

@m1kola m1kola enabled auto-merge June 17, 2024 15:41
@m1kola m1kola added this pull request to the merge queue Jun 17, 2024
Copy link
Member

@varshaprasad96 varshaprasad96 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

/lgtm
/approve

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 17, 2024
Merged via the queue into operator-framework:main with commit 8d38db4 Jun 17, 2024
10 checks passed
@m1kola m1kola deleted the test_limitations branch June 18, 2024 08:17
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm Indicates that a PR is ready to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants