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

feat: accepted policy status condition #347

Merged
merged 13 commits into from
Jan 9, 2024

refactor: interface for standard policy errors

6874ca6
Select commit
Loading
Failed to load commit list.
Merged

feat: accepted policy status condition #347

refactor: interface for standard policy errors
6874ca6
Select commit
Loading
Failed to load commit list.
Codecov / codecov/patch succeeded Jan 9, 2024 in 0s

89.36% of diff hit (target 65.01%)

View this Pull Request on Codecov

89.36% of diff hit (target 65.01%)

Annotations

Check warning on line 283 in api/v1beta2/authpolicy_types.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

api/v1beta2/authpolicy_types.go#L282-L283

Added lines #L282 - L283 were not covered by tests

Check warning on line 246 in api/v1beta2/ratelimitpolicy_types.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

api/v1beta2/ratelimitpolicy_types.go#L245-L246

Added lines #L245 - L246 were not covered by tests

Check warning on line 141 in controllers/authpolicy_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/authpolicy_controller.go#L140-L141

Added lines #L140 - L141 were not covered by tests

Check warning on line 160 in controllers/ratelimitpolicy_controller.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

controllers/ratelimitpolicy_controller.go#L159-L160

Added lines #L159 - L160 were not covered by tests

Check warning on line 169 in pkg/reconcilers/targetref_reconciler.go

See this annotation in the file changed.

@codecov codecov / codecov/patch

pkg/reconcilers/targetref_reconciler.go#L169

Added line #L169 was not covered by tests