Skip to content

Adds support for the 'in' filter to slcli call-api#728

Merged
sudorandom merged 8 commits into
softlayer:masterfrom
sudorandom:call-api-in-filter
Aug 24, 2016
Merged

Adds support for the 'in' filter to slcli call-api#728
sudorandom merged 8 commits into
softlayer:masterfrom
sudorandom:call-api-in-filter

Conversation

@sudorandom
Copy link
Copy Markdown
Contributor

No description provided.

@sudorandom
Copy link
Copy Markdown
Contributor Author

Resolves #727

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.01%) to 79.802% when pulling 3e58fc9 on sudorandom:call-api-in-filter into def654a on softlayer:master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.01%) to 79.802% when pulling 3e58fc9 on sudorandom:call-api-in-filter into def654a on softlayer:master.

@thekuffs
Copy link
Copy Markdown

lgtm :shipit:

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.01%) to 79.802% when pulling 24b7011 on sudorandom:call-api-in-filter into def654a on softlayer:master.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.01%) to 79.802% when pulling 24b7011 on sudorandom:call-api-in-filter into def654a on softlayer:master.

@camporter
Copy link
Copy Markdown
Member

A more backwards-compatible way of changing this would be to add a new keyword. The docs for the _build_filters method says "This only supports the equals keyword at the moment", which seems to indicate that additional keywords would be added in the future.

There could be cases, such as various names on objects (images, ssh keys, etc), that may have commas in them.

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage increased (+0.04%) to 79.836% when pulling f09debf on sudorandom:call-api-in-filter into 62515bd on softlayer:master.

@sudorandom sudorandom added this to the v5.2.0 milestone Aug 24, 2016
@sudorandom sudorandom merged commit 29d86d3 into softlayer:master Aug 24, 2016
@sudorandom sudorandom deleted the call-api-in-filter branch August 24, 2016 03:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants