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.

Sign up for GitHub

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

Jump to bottom

refactor: make remaining rules implement rule interface #131

Merged
merged 1 commit into from
Aug 23, 2024
Merged

refactor: make remaining rules implement rule interface #131

merged 1 commit into from
Aug 23, 2024

Conversation

mattwelke
Copy link
Member

Description

Forgot two of the four rules when doing #130. This does the rest.

@mattwelke mattwelke requested a review from a team as a code owner August 23, 2024 00:11
@dosubot dosubot bot added the size:M This PR changes 30-99 lines, ignoring generated files. label Aug 23, 2024
@dosubot dosubot bot added the enhancement Enhancement to an existing feature label Aug 23, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 23, 2024
@codecov Codecov
Copy link

codecov bot commented Aug 23, 2024 •
edited
Loading

Codecov Report

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

Files Patch % Lines
api/v1alpha1/maasvalidator_types.go 50.00% 4 Missing ⚠️ 
@@            Coverage Diff             @@
##             main     #131      +/-   ##
==========================================
- Coverage   74.78%   74.22%   -0.56%     
==========================================
  Files           9        9              
  Lines         349      357       +8     
==========================================
+ Hits          261      265       +4     
- Misses         72       76       +4     
  Partials       16       16
Files Coverage Δ
pkg/validators/dns/upstream_dns_validator.go 92.30% <100.00%> (ø)
pkg/validators/res/resource_validator.go 83.33% <100.00%> (ø)
api/v1alpha1/maasvalidator_types.go 76.92% <50.00%> (-11.97%) ⬇️

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 259508a...aca838b. Read the comment docs.

@ahmad-ibra ahmad-ibra merged commit bb370f0 into main Aug 23, 2024
7 of 8 checks passed
@ahmad-ibra ahmad-ibra deleted the refactor/validation-rule-interface-remaining-rules branch August 23, 2024 00:20
@TylerGillson TylerGillson mentioned this pull request Aug 23, 2024
Merged
mattwelke pushed a commit that referenced this pull request Aug 23, 2024
@TylerGillson
chore(main): release 0.0.9 (#132)
58a1f50 
🤖 I have created a release *beep* *boop*
---


##
[0.0.9](v0.0.8...v0.0.9)
(2024-08-23)


### Refactoring

* make remaining rules implement rule interface
([#131](#131))
([bb370f0](bb370f0))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
ahmad-ibra pushed a commit that referenced this pull request Aug 23, 2024
@mattwelke @ahmad-ibra
refactor: make remaining rules implement rule interface (#131)
ed0bc5b 
## Description
Forgot two of the four rules when doing
#130. This
does the rest.

Signed-off-by: Matt Welke <[email protected]>
@TylerGillson TylerGillson mentioned this pull request Aug 23, 2024
Merged
mattwelke pushed a commit that referenced this pull request Aug 23, 2024
@TylerGillson
chore(main): release 0.0.8 (#138)
0c515c8 
🤖 I have created a release *beep* *boop*
---


##
[0.0.8](v0.0.7...v0.0.8)
(2024-08-23)


### Bug Fixes

* embedding structs related to `validationrule.Interface`
([#135](#135))
([4ebf38a](4ebf38a))


### Other

* assert that PluginSpec is implemented
([#133](#133))
([cf2c23f](cf2c23f))
* satisfy ValidationRule
([#121](#121))
([5894f51](5894f51))


### Dependency Updates

* **deps:** update module github.com/canonical/gomaasclient to v0.7.0
([4942f99](4942f99))
* **deps:** update module github.com/onsi/ginkgo/v2 to v2.20.0
([951e094](951e094))
* **deps:** update module github.com/onsi/ginkgo/v2 to v2.20.1
([cab9803](cab9803))
* **deps:** update module github.com/validator-labs/validator to v0.1.2
([df63d44](df63d44))
* **deps:** update module github.com/validator-labs/validator to v0.1.5
([#123](#123))
([a856370](a856370))
* **deps:** update module github.com/validator-labs/validator to v0.1.6
([5dbc31f](5dbc31f))
* **deps:** update module sigs.k8s.io/controller-runtime to v0.19.0
([#124](#124))
([e903cc7](e903cc7))


### Refactoring

* make each rule implement `validationrule.Interface`
([#130](#130))
([381e3ab](381e3ab))
* make remaining rules implement rule interface
([#131](#131))
([ed0bc5b](ed0bc5b))
* support direct rule evaluation
([#118](#118))
([e4d873d](e4d873d))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
@TylerGillson TylerGillson mentioned this pull request Aug 23, 2024
Merged
TylerGillson added a commit that referenced this pull request Aug 23, 2024
@TylerGillson
chore(main): release 0.0.8 (#140)
8d182d2 
🤖 I have created a release *beep* *boop*
---


##
[0.0.8](v0.0.7...v0.0.8)
(2024-08-23)


### Bug Fixes

* embedding structs related to `validationrule.Interface`
([#135](#135))
([4ebf38a](4ebf38a))


### Other

* assert that PluginSpec is implemented
([#133](#133))
([cf2c23f](cf2c23f))
* satisfy ValidationRule
([#121](#121))
([5894f51](5894f51))


### Dependency Updates

* **deps:** update module github.com/canonical/gomaasclient to v0.7.0
([4942f99](4942f99))
* **deps:** update module github.com/onsi/ginkgo/v2 to v2.20.0
([951e094](951e094))
* **deps:** update module github.com/onsi/ginkgo/v2 to v2.20.1
([cab9803](cab9803))
* **deps:** update module github.com/validator-labs/validator to v0.1.2
([df63d44](df63d44))
* **deps:** update module github.com/validator-labs/validator to v0.1.5
([#123](#123))
([a856370](a856370))
* **deps:** update module github.com/validator-labs/validator to v0.1.6
([5dbc31f](5dbc31f))
* **deps:** update module sigs.k8s.io/controller-runtime to v0.19.0
([#124](#124))
([e903cc7](e903cc7))


### Refactoring

* make each rule implement `validationrule.Interface`
([#130](#130))
([381e3ab](381e3ab))
* make remaining rules implement rule interface
([#131](#131))
([ed0bc5b](ed0bc5b))
* support direct rule evaluation
([#118](#118))
([e4d873d](e4d873d))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).
@TylerGillson TylerGillson mentioned this pull request Aug 24, 2024
Closed
@mattwelke mattwelke mentioned this pull request Aug 24, 2024
Closed
@TylerGillson TylerGillson mentioned this pull request Aug 24, 2024
Merged
mattwelke pushed a commit that referenced this pull request Aug 24, 2024
@TylerGillson
chore(main): release 0.0.11 (#150)
3b8666b 
🤖 I have created a release *beep* *boop*
---


##
[0.0.11](v0.0.11...v0.0.11)
(2024-08-24)


### Other

* add retract directive for versions v0.0.8-v0.0.10, release 0.0.11
([#147](#147))
([27e1988](27e1988))
* assert that PluginSpec is implemented
([#133](#133))
([cf2c23f](cf2c23f))
* manually set version to 0.0.11 for release-please
([#145](#145))
([f26f1fa](f26f1fa))
* release 0.0.11
([#149](#149))
([297bb73](297bb73))


### Dependency Updates

* **deps**: update module github.com/validator-labs/validator to v0.1.8
* **deps:** update module github.com/canonical/gomaasclient to v0.7.0
([4942f99](4942f99))
* **deps:** update module github.com/validator-labs/validator to v0.1.2
([df63d44](df63d44))
* **deps:** update module github.com/validator-labs/validator to v0.1.5
([#123](#123))
([a856370](a856370))
* **deps:** update module github.com/validator-labs/validator to v0.1.6
([5dbc31f](5dbc31f))
* **deps:** update module sigs.k8s.io/controller-runtime to v0.19.0
([#124](#124))
([e903cc7](e903cc7))


### Refactoring

* make each rule implement `validationrule.Interface`
* initial PR - make each rule implement `validationrule.Interface`
([#130](#130))
([381e3ab](381e3ab))
* bug fix 1 - embedding structs related to `validationrule.Interface`
([#135](#135))
([4ebf38a](4ebf38a))
* bug fix 2 - YAML rendering of structs embedded in rules
([#143](#143))
([c7ae66d](c7ae66d))
* make remaining rules implement rule interface
([#131](#131))
([ed0bc5b](ed0bc5b))
* support direct rule evaluation
([#118](#118))
([e4d873d](e4d873d))

---
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
Reviewers

@ahmad-ibra ahmad-ibra ahmad-ibra approved these changes

@arturshadnik arturshadnik Awaiting requested review from arturshadnik arturshadnik is a code owner automatically assigned from validator-labs/validator

Assignees

@ahmad-ibra ahmad-ibra

Labels
enhancement Enhancement to an existing feature lgtm This PR has been approved by a maintainer size:M This PR changes 30-99 lines, ignoring generated files.
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@mattwelke @ahmad-ibra