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: ensure unique condition.ValidationRule names across different rule types #377

Merged
merged 3 commits into from
Oct 3, 2024

Conversation

ahmad-ibra
Copy link
Contributor

Issue

Resolves #376

Description

This PR ensures that the validation rule type is a part of the validation results rule name.

@ahmad-ibra ahmad-ibra requested a review from a team as a code owner October 3, 2024 21:52
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Oct 3, 2024
mattwelke
mattwelke previously approved these changes Oct 3, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Oct 3, 2024
Copy link

codecov bot commented Oct 3, 2024

Codecov Report

Attention: Patch coverage is 77.77778% with 4 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
pkg/validators/ntp/ntp.go 0.00% 4 Missing ⚠️
@@            Coverage Diff             @@
##             main     #377      +/-   ##
==========================================
+ Coverage   39.48%   39.55%   +0.07%     
==========================================
  Files          32       32              
  Lines        2001     2005       +4     
==========================================
+ Hits          790      793       +3     
- Misses       1111     1112       +1     
  Partials      100      100              
Files with missing lines Coverage Δ
...kg/validators/computeresources/computeresources.go 76.76% <100.00%> (+0.11%) ⬆️
pkg/validators/privileges/privilege.go 100.00% <100.00%> (ø)
pkg/validators/tags/tags.go 75.34% <100.00%> (+0.34%) ⬆️
pkg/validators/ntp/ntp.go 17.85% <0.00%> (-0.67%) ⬇️

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c44ce5d...8e90551. Read the comment docs.

@ahmad-ibra ahmad-ibra merged commit 2a6f5ab into main Oct 3, 2024
8 checks passed
@ahmad-ibra ahmad-ibra deleted the fix/unique-condition-rulename branch October 3, 2024 22:14
ahmad-ibra pushed a commit that referenced this pull request Oct 3, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.1.4](v0.1.3...v0.1.4)
(2024-10-03)


### Bug Fixes

* ensure unique condition.ValidationRule names across different rule
types
([#377](#377))
([2a6f5ab](2a6f5ab))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

🐛 Duplicate condition.ValidationRule names are currently possible
2 participants