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: validate rules in loop #103

Merged
merged 2 commits into from Jul 1, 2021
Merged

fix: validate rules in loop #103

merged 2 commits into from Jul 1, 2021

Conversation

ankithans
Copy link
Member

@ankithans ankithans commented Jul 1, 2021

Closes #98
Closes #93
Closes #53

Description

  • Rules implement Rule interface
  • Validate rules in for loop(DRY), instead of calling one by one
  • CI/CD tests & builds validator example

Type of change

  • Bug fix (non-breaking change which fixes an issue)

@ankithans ankithans added the bug Something isn't working label Jul 1, 2021
@ankithans ankithans requested a review from wtrocki July 1, 2021 04:25
@ankithans ankithans self-assigned this Jul 1, 2021
@@ -10,6 +10,10 @@ import (
"github.com/spf13/cobra"
)

type Rules interface {
Copy link
Contributor

Choose a reason for hiding this comment

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

doc

@@ -10,6 +10,10 @@ import (
"github.com/spf13/cobra"
)

type Rules interface {
Validate() []validator.ValidationError
Copy link
Contributor

Choose a reason for hiding this comment

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

Wrap error aray in result structure for other elements

Copy link
Contributor

@wtrocki wtrocki Jul 1, 2021

Choose a reason for hiding this comment

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

actually lets leave it likt this

@wtrocki wtrocki merged commit ca6b5ce into main Jul 1, 2021
@wtrocki wtrocki deleted the ankithans/issue98 branch July 1, 2021 07:15
@wtrocki
Copy link
Contributor

wtrocki commented Jul 1, 2021

Lets release this /create github release v0.1.0

@ankithans
Copy link
Member Author

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Rule methods should implement interface CI/CD support for validator Setup CI/CD to the repository
2 participants