-
Notifications
You must be signed in to change notification settings - Fork 72
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
Relocate format checkers #161
Relocate format checkers #161
Conversation
💚 Build Succeeded
Expand to view the summary
Build stats
Trends 🧪 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is everything alright with this PR? I see it includes changes introduced in 92ee6e4 . Is that intended?
4cdc60a
to
50c5034
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It might be confusing for users that this semantic validator requires special handling - Load/Unload. I would be for adjusting it to the concept func do() validationErrors
, but I don't think it's easy. Let's merge this one!
@@ -8,6 +8,8 @@ import ( | |||
"regexp" | |||
"sync" | |||
|
|||
"github.com/elastic/package-spec/code/go/internal/validator/semantic" | |||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nit: sorting imports
What does this PR do?
This PR relocates the format checkers implementation into the new
semantic
package.Why is it important?
With this change, all non-declarative package validation rules are contained in a single place, the
semantic
package. This will make porting such rules to another language easier.Checklist
I have added test packages toTest packages already exist.test/packages
that prove my change is effective.I have added an entry inNo changes to spec.versions/N/changelog.yml
.Related issues