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

chore: clean up helpers for setting up oci network and vsphere validation rules #137

Merged
merged 4 commits into from
Aug 7, 2024

Conversation

ahmad-ibra
Copy link
Collaborator

Description

These helpers are no longer required as we can now easily invoke validator check in --direct mode

@ahmad-ibra ahmad-ibra force-pushed the feat/add-configure-validation-helpers branch from 9771d46 to c0b5a22 Compare August 7, 2024 00:26
@ahmad-ibra ahmad-ibra changed the title feat: clean up helpers for setting up oci network and vsphere validation rules chore: clean up helpers for setting up oci network and vsphere validation rules Aug 7, 2024
@ahmad-ibra ahmad-ibra force-pushed the feat/add-configure-validation-helpers branch from c0b5a22 to 67e30e2 Compare August 7, 2024 00:33
Copy link

codecov bot commented Aug 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

@@            Coverage Diff             @@
##             main     #137      +/-   ##
==========================================
+ Coverage   37.60%   38.34%   +0.74%     
==========================================
  Files          44       41       -3     
  Lines        5435     5330     -105     
==========================================
  Hits         2044     2044              
+ Misses       3005     2900     -105     
  Partials      386      386              
Files Coverage Δ
pkg/components/validator.go 58.07% <ø> (+3.23%) ⬆️

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 54aa49f...67e30e2. Read the comment docs.

@ahmad-ibra ahmad-ibra marked this pull request as ready for review August 7, 2024 01:37
@ahmad-ibra ahmad-ibra requested a review from a team as a code owner August 7, 2024 01:37
@dosubot dosubot bot added size:L This PR changes 100-499 lines, ignoring generated files. refactoring Refactoring / tech debt labels Aug 7, 2024
@dosubot dosubot bot added the lgtm This PR has been approved by a maintainer label Aug 7, 2024
@ahmad-ibra ahmad-ibra merged commit dbdee2f into main Aug 7, 2024
8 checks passed
@ahmad-ibra ahmad-ibra deleted the feat/add-configure-validation-helpers branch August 7, 2024 16:21
ahmad-ibra pushed a commit that referenced this pull request Aug 8, 2024
🤖 I have created a release *beep* *boop*
---


##
[0.1.1](v0.1.0...v0.1.1)
(2024-08-08)


### Other

* clean up helpers for setting up oci network and vsphere validation
rules
([#137](#137))
([dbdee2f](dbdee2f))


### Docs

* update docs to reflect recent api changes
([#146](#146))
([f2b3217](f2b3217))


### Dependency Updates

* **deps:** update github.com/validator-labs/validator-plugin-azure
digest to 862db62
([#142](#142))
([039e6f0](039e6f0))
* **deps:** update github.com/validator-labs/validator-plugin-vsphere
digest to d7deabd
([#143](#143))
([72267ad](72267ad))
* **deps:** update module github.com/validator-labs/validator-plugin-oci
to v0.1.0
([#132](#132))
([8862bb5](8862bb5))


### Refactoring

* add rules subcommand & split out apply/check
([#144](#144))
([e88bd71](e88bd71))

---
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 refactoring Refactoring / tech debt size:L This PR changes 100-499 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants