Skip to content

targetEnforcementAction key should take care of the object kind#510

Merged
ritazh merged 1 commit intoopen-policy-agent:masterfrom
WnP:target-enforcement-action-key
Jan 15, 2025
Merged

targetEnforcementAction key should take care of the object kind#510
ritazh merged 1 commit intoopen-policy-agent:masterfrom
WnP:target-enforcement-action-key

Conversation

@WnP
Copy link
Copy Markdown
Contributor

@WnP WnP commented Jan 10, 2025

Signed-off-by: Steeve Chailloux <me@steeve.io>
Copy link
Copy Markdown
Contributor

@JaydipGabani JaydipGabani left a comment

Choose a reason for hiding this comment

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

Thanks for PR! LGTM

Copy link
Copy Markdown
Contributor

@maxsmythe maxsmythe left a comment

Choose a reason for hiding this comment

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

LGTM, thanks for fixing this!

@codecov-commenter
Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 53.44%. Comparing base (76869f8) to head (0a9a1d2).
Report is 42 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #510      +/-   ##
==========================================
- Coverage   54.68%   53.44%   -1.25%     
==========================================
  Files          71       97      +26     
  Lines        5241     6117     +876     
==========================================
+ Hits         2866     3269     +403     
- Misses       2073     2513     +440     
- Partials      302      335      +33     
Flag Coverage Δ
unittests 53.44% <100.00%> (-1.25%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

Copy link
Copy Markdown
Member

@ritazh ritazh left a comment

Choose a reason for hiding this comment

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

Thanks for the fix!!!

@JaydipGabani need to bump framework in GK.

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.

Gator does not return the same status code for the same given context

5 participants