-
Notifications
You must be signed in to change notification settings - Fork 479
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 aci_connector addon #230
Conversation
MAIN BRANCH PUSH DETECTED DUE TO #241, THIS PR WILL BE UPDATED. |
MAIN BRANCH PUSH DETECTED DUE TO #, THIS PR WILL BE UPDATED. |
1 similar comment
MAIN BRANCH PUSH DETECTED DUE TO #, THIS PR WILL BE UPDATED. |
MAIN BRANCH PUSH DETECTED DUE TO #, THIS PR NEED TO BE UPDATED TO TRIGGER CI. |
Hi @zioproto , I'd like to review this pr since now we have CI pipeline, would you please solve the conflict? Thanks! |
60e3663
to
15836de
Compare
MAIN BRANCH PUSH DETECTED DUE TO #, THIS PR NEED TO BE UPDATED TO TRIGGER CI. |
15836de
to
73f06ab
Compare
@lonegunmanb all tests are passing, I believe this one is ready to merge. 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.
LGTM, thanks @zioproto
This is a refactor of pr #130 on the new version of the module.