-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
🌱 Add Template compatibility checks to topology/checks package #5641
🌱 Add Template compatibility checks to topology/checks package #5641
Conversation
c54fc3b
to
c8a5b12
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.
Looks good overall, a few nits
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.
Overall lgtm apart some nit on naming / on avoiding duplicate logic
21b22fa
to
1da04b8
Compare
1da04b8
to
75ac24a
Compare
75ac24a
to
655f9bc
Compare
|
/retest |
c81ddbe
to
fb4674a
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.
/lgtm
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.
Mostly nits
fb4674a
to
248c63f
Compare
Signed-off-by: killianmuldoon <kmuldoon@vmware.com> fix naming issues etc.
248c63f
to
ffc0379
Compare
/lgtm |
Great work! |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: fabriziopandini The full list of commands accepted by this bot can be found here. The pull request process is described here
Needs approval from an approver in each of these files:
Approvers can indicate their approval by writing |
Signed-off-by: killianmuldoon kmuldoon@vmware.com
Add Template compatibility check functions to the Topology/checks package so they can be reused.