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: add AvroValidationRule for names and symbols #197

Merged
merged 2 commits into from
Oct 25, 2024

Conversation

VisualBean
Copy link
Collaborator

This pull request introduces several changes to the LEGO.AsyncAPI project, primarily focusing on enhancing validation capabilities and adding support for Avro schema payloads. The most important changes include adding new validation rules, updating resource strings, and extending the visitor and walker functionalities.

Validation Enhancements:

  • Added new validation rules for AsyncApiAvroSchemaPayload to ensure names and symbols match specified regular expressions. (src/LEGO.AsyncAPI/Validation/Rules/AsyncApiAvroRules.cs)
  • Introduced new resource strings for validation messages related to name and symbol regular expressions. (src/LEGO.AsyncAPI/Resource.Designer.cs, src/LEGO.AsyncAPI/Resource.resx) [1] [2]

Visitor and Walker Extensions:

  • Added Visit methods for AsyncApiJsonSchemaPayload and AsyncApiAvroSchemaPayload in AsyncApiVisitorBase. (src/LEGO.AsyncAPI/Services/AsyncApiVisitorBase.cs)
  • Extended the AsyncApiWalker to handle AsyncApiAvroSchemaPayload. (src/LEGO.AsyncAPI/Services/AsyncApiWalker.cs) [1] [2]

Validation Integration:

  • Updated AsyncApiValidator to include validation for AsyncApiAvroSchemaPayload and AsyncApiJsonSchemaPayload. (src/LEGO.AsyncAPI/Validation/AsyncApiValidator.cs)

@VisualBean VisualBean merged commit f7d96d8 into vnext Oct 25, 2024
5 checks passed
@VisualBean VisualBean deleted the alex/avrovalidation branch October 25, 2024 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant