-
Notifications
You must be signed in to change notification settings - Fork 450
refactor(parser): Improve VPC Lattice with examples and descriptions #7234
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
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #7234 +/- ##
========================================
Coverage 96.35% 96.35%
========================================
Files 275 275
Lines 12980 12980
Branches 965 965
========================================
Hits 12507 12507
Misses 366 366
Partials 107 107 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
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.
Hey @anafalcao! Please remove the 2 references to IP Address. Since this code goes to the customer codebase, it can trigger some security alerts.
|
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.
Thanks for working on this @anafalcao! APPROVED
Issue number: closes #7101
Summary
Enhances the VPC Lattice parser models with field descriptions and examples using Pydantic's Field() functionality. This improvement provides better documentation and metadata for VPC Lattice event parsing, following the pattern established in other AWS event models.
This change is fully backward compatible - no breaking changes to existing validation and serialization process.
User experience
Customers will be able to see examples and description when serializing models.
Checklist
If your change doesn't seem to apply, please leave them unchecked.
Is this a breaking change?
RFC issue number:
Checklist:
Acknowledgment
By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.
Disclaimer: We value your time and bandwidth. As such, any pull requests created on non-triaged issues might not be successful.