-
Notifications
You must be signed in to change notification settings - Fork 62
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
Added editorial style guide checking. #620
Conversation
Changes AnalysisCommit SHA: 93c1b65 API ChangesSummaryNO CHANGES ReportThe full API changes report is available at: https://github.com/opensearch-project/opensearch-api-specification/actions/runs/11507315122/artifacts/2101233351 API Coverage
|
8d9f61e
to
e22d550
Compare
Spec Test Coverage Analysis
|
efba839
to
a7a7991
Compare
@kolchfa-aws please take a look, this is ready to be merged |
87b7199
to
2cd1f8a
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.
I noted that we can remove several rules from here because they are not needed. Otherwise, LGTM
@nhtruong care to CR when you get a chance, please? |
Signed-off-by: dblock <[email protected]>
Signed-off-by: dblock <[email protected]>
Signed-off-by: dblock <[email protected]>
Signed-off-by: Theo N. Truong <[email protected]>
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.
The ability to pinpoint exactly where the error happens with git diff is so nice. We should add this to the spec linter in the future!
Description
Add support for running Vale on specs. This gets us reviews of .yaml files being changed.
Issues Resolved
Closes #599.
By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on following Developer Certificate of Origin and signing off your commits, please check here.