-
Notifications
You must be signed in to change notification settings - Fork 131
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
[RFC-0006] Add CDEvent Receiver Support #772
Conversation
@adamkenihan can you please signoff all your commits, this is a requirement for Flux contributions. Also can you run |
2f39de6
to
d70bcf8
Compare
This is almost ready. Please rebase on the latest main branch. I think we'll be able to merge it soon. |
The git worktree seems to be in a dirty state after running the CI build steps, see https://github.com/fluxcd/notification-controller/actions/runs/8423005933/job/23075692347?pr=772. |
d70bcf8
to
89b82e4
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! Thanks.
Left a minor suggestion which can used or ignored.
Signed-off-by: adamkenihan <adam.kenihan@est.tech> Added CDEvents Receiver Signed-off-by: adamkenihan <adam.kenihan@est.tech> Added CDEvent Validation + Tests Signed-off-by: adamkenihan <adam.kenihan@est.tech> Small changes to CDEvent Receiver Signed-off-by: adamkenihan <adam.kenihan@est.tech> More CDEvents Tests and Docs added Signed-off-by: adamkenihan <adam.kenihan@est.tech> Adding imports to go.mod Signed-off-by: adamkenihan <adam.kenihan@est.tech> small changes + manifests and formatting Signed-off-by: adamkenihan <adam.kenihan@est.tech> Changing string compare to EqualFold Signed-off-by: adamkenihan <adam.kenihan@est.tech>
89b82e4
to
b4949b6
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
Thanks @adamkenihan 🥇
Adding CDEvent Receiver implementation according to RFC-0006
Fix: #769