-
Notifications
You must be signed in to change notification settings - Fork 141
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 Middleware API #255
Conversation
Checked commit cfcosta@cab292b with ruby 2.3.3, rubocop 0.47.1, haml-lint 0.20.0, and yamllint 1.10.0 |
@miq-bot add_label providers/middleware |
@cfcosta if this is an "as is" revert of the previous commit that added the api, it's better (and more clear) to send a regular revert |
@abonas not exactly, it is a revert of the whole PR merge. It's the cleanest way of doing it, but it squashes all commits from the PR into one. |
OK thanks |
test failures seem unrelated from a quick glance |
ping @cfcosta when you get a chance to push the git revert commit instead, Thanks!! |
cab292b
to
294567f
Compare
@imtayadeway @abellotti it was not done by hand, I did the |
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.
Thanks @cfcosta !
Remove Middleware API (cherry picked from commit f4f2ca7)
Gaprindashvili backport details:
|
Middleware Manager is being phased out, so is the API.
@miq-bot add_label gaprindashvili/yes