-
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
bluecat: add delete functionality #2144
bluecat: add delete functionality #2144
Conversation
Welcome @jaideepbellani! |
/assign |
/assgin @vinny-sabatini |
/assign @vinny-sabatini |
/kind bug |
b2f8e5a
to
d62aa68
Compare
dc3b28a
to
ccd6a13
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.
Nice work! I added some small suggestions inline.
/lgtm |
@vinny-sabatini this change looks good to me. Please add your approval if you think this looks good. Adding a hold so commits can be squashed before merging. |
/lgtm |
@vinny-sabatini the requested changes have been made. Please review one more time. Thanks! |
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 to me, nice job @jaideepbellani !
Please cleanup your commit history and we can get this merged
/approve |
@jaideepbellani please squash commits and we will merge it. Thanks! |
a19be9a
to
a0f7f95
Compare
a0f7f95
to
1a3e6af
Compare
1a3e6af
to
f2f8681
Compare
/lgtm |
[APPROVALNOTIFIER] This PR is APPROVED This pull-request has been approved by: jaideepbellani, seanmalloy, vinny-sabatini 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 |
Description
Added the ability to delete records for the Bluecat provider
Fixes #2143
Checklist