Skip to content
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 support for *List kinds #213

Merged
merged 7 commits into from
Oct 4, 2018
Merged

Conversation

fabianvf
Copy link
Member

Should resolve #175 #209

Relates to ansible/ansible#42558

@openshift-ci-robot openshift-ci-robot added the size/M Denotes a PR that changes 30-99 lines, ignoring generated files. label Sep 26, 2018
@fabianvf
Copy link
Member Author

ping @willthames, first stab at *List support.

@willthames
Copy link
Contributor

Looks good so far.

@fabianvf
Copy link
Member Author

So this seems to be working properly, other than tests (which I'm working on now), is there anything you would change? Once this goes into a release I'm hoping for a PR to Ansible to get rid of any special casing for lists, I think they should be able to drop in seamlessly.

@openshift-ci-robot openshift-ci-robot added size/L Denotes a PR that changes 100-499 lines, ignoring generated files. and removed size/M Denotes a PR that changes 30-99 lines, ignoring generated files. labels Sep 27, 2018
@openshift-ci-robot openshift-ci-robot added size/XL Denotes a PR that changes 500-999 lines, ignoring generated files. and removed size/L Denotes a PR that changes 100-499 lines, ignoring generated files. labels Oct 2, 2018
@fabianvf fabianvf merged commit 89b5e8e into openshift:master Oct 4, 2018
fabianvf added a commit to fabianvf/openshift-restclient-python that referenced this pull request Nov 5, 2018
* Add support for *List kinds
* Update  tests to support *List kinds

(cherry picked from commit 89b5e8e)
willthames pushed a commit to willthames/openshift-restclient-python that referenced this pull request Nov 22, 2018
* Add support for *List kinds
* Update  tests to support *List kinds
willthames pushed a commit to willthames/openshift-restclient-python that referenced this pull request Nov 22, 2019
* Add support for *List kinds
* Update  tests to support *List kinds
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XL Denotes a PR that changes 500-999 lines, ignoring generated files.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dynamic Client should handle Lists
3 participants