-
Notifications
You must be signed in to change notification settings - Fork 2.6k
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
Remove PropertyValuesEqual method from Provider interface #3702
Remove PropertyValuesEqual method from Provider interface #3702
Conversation
d2cfcbc
to
c4eae81
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.
@johngmyers In general this looks good to me. One question I have is how can we make sure that we spot any regressions with this change. Do you have access to some of the providers impacted, for example cloudflare?
I do not have access to Cloudflare. To spot AWS regressions I added to the AWS tests assertions that The Cloudflare unit tests do not make assertions about the content of returned records, just their expected number. So I did not attempt to adjust those tests. |
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.
@johngmyers a couple more comments, then we're good to go.
c4eae81
to
82b0c70
Compare
82b0c70
to
70f60d7
Compare
70f60d7
to
5affab0
Compare
/approve |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: johngmyers, mloiseleur, Raffo 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 |
/retest |
Description
This is prep work for the webhook provider. Removes the frequently-called
PropertyValuesEqual
method from theProvider
andRegistry
interfaces, requiring providers to normalize provider-specific properties in theirAdjustEndpoints
method.Checklist
End user documentation updated