fix. sync webhookconfiguration from manifest#170
fix. sync webhookconfiguration from manifest#170Colvin-Y wants to merge 1 commit intoopenkruise:masterfrom
Conversation
|
[APPROVALNOTIFIER] This PR is NOT APPROVED This pull-request has been approved by: The full list of commands accepted by this bot can be found here. DetailsNeeds approval from an approver in each of these files:Approvers can indicate their approval by writing |
|
Welcome @Colvin-Y! It looks like this is your first PR to openkruise/charts 🎉 |
furykerry
left a comment
There was a problem hiding this comment.
i think the updated webhook should be placed in the version called 'next', not 1.8.3, i can copy the changes to 1.9
ack, I'll change the target branch and dir. |
Signed-off-by: Colvin-Y <ykwhrimfaxi@gmail.com>
bbe7e51 to
78c5f45
Compare
Fixed. |
fix #169
Note on DCO:
If the DCO action in the integration test fails, one or more of your commits are not signed off. Please click on the Details link next to the DCO action for instructions on how to resolve this.
Checklist:
Changes are automatically published when merged to
master. They are not published on branches.