Skip to content
This repository has been archived by the owner on May 23, 2024. It is now read-only.

Default API version to 1 #113

Closed
wants to merge 1 commit into from
Closed

Conversation

jameswnl
Copy link
Contributor

The major consumer of this gem is manageiq-providers-ansible_tower and which is based on v1 api. Before manageiq-providers-ansible_tower support v2 in, we shall keep this gem. And besides currently only the credential_type from v2 is supported in this gem.

@jameswnl
Copy link
Contributor Author

@bdunne Please review

@miq-bot
Copy link
Collaborator

miq-bot commented Jul 16, 2018

Checked commit jameswnl@904cf81 with ruby 2.3.3, rubocop 0.52.1, haml-lint 0.20.0, and yamllint 1.10.0
2 files checked, 0 offenses detected
Everything looks fine. 🍪

@bdunne bdunne mentioned this pull request Jul 27, 2018
@jameswnl
Copy link
Contributor Author

#117 is reverting the PRs. closing this.

@jameswnl jameswnl closed this Jul 27, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants