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

Refactor endpoint #64

Merged
merged 3 commits into from
Jan 25, 2017
Merged

Refactor endpoint #64

merged 3 commits into from
Jan 25, 2017

Conversation

syncrou
Copy link
Contributor

@syncrou syncrou commented Jan 24, 2017

Pulling specific self.endpoint methods out of all the subclasses and standardize it from the base_model

@syncrou
Copy link
Contributor Author

syncrou commented Jan 24, 2017

@miq-bot assign @bdunne

@miq-bot add_label refactor

@miq-bot
Copy link
Collaborator

miq-bot commented Jan 24, 2017

@syncrou Cannot apply the following label because they are not recognized: refactor

@syncrou
Copy link
Contributor Author

syncrou commented Jan 24, 2017

@miq-bot add_label enhancement

@miq-bot
Copy link
Collaborator

miq-bot commented Jan 25, 2017

Checked commits syncrou/ansible_tower_client@f393f43~...9765a2b with ruby 2.2.6, rubocop 0.47.1, and haml-lint 0.20.0
22 files checked, 0 offenses detected
Everything looks good. 🏆

@bdunne bdunne merged commit c3fff02 into ansible:master Jan 25, 2017
@syncrou syncrou deleted the refactor_endpoint branch January 25, 2017 15:45
bdunne added a commit that referenced this pull request Feb 2, 2017
Added
- Refactor `#endpoint` [(#64)](#64)
- Expose playbooks off of projects [(#62)](#62)

Changed
- Allow for alternative resource paths [(#66)](#66)
- Raise UnlicensedFeatureError when we receive HTTP 402 [(#65)](#65)

Fixed
- Adjusted project_spec to test on a Project [(#63)](#63)
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants