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

feat(YAML): allows using lists or single values with arg declarations #620

Merged
merged 1 commit into from Aug 20, 2016

Conversation

kbknapp
Copy link
Member

@kbknapp kbknapp commented Aug 20, 2016

One can now use a list or single value for certain Arg YAML declarations
such as possible_values, etc.

Prior to this commit, if only a single value was desired one would have
to use the format:

possible_values:
    - value

But now once can use

possible_values: value

Closes #614
Closes #613

@yo-bot
Copy link

yo-bot commented Aug 20, 2016

r? @sru

(yo-bot has picked a reviewer for you, use r? to override)

One can now use a list or single value for certain Arg YAML declarations
such as possible_values, etc.

Prior to this commit, if only a single value was desired one would have
to use the format:

```yaml
possible_values:
    - value
```

But now once can use

```yaml
possible_values: value
```

Closes #614
Closes #613
@kbknapp
Copy link
Member Author

kbknapp commented Aug 20, 2016

@homu r+

@homu
Copy link
Contributor

homu commented Aug 20, 2016

📌 Commit 9ade2cd has been approved by kbknapp

homu added a commit that referenced this pull request Aug 20, 2016
feat(YAML): allows using lists or single values with arg declarations

One can now use a list or single value for certain Arg YAML declarations
such as possible_values, etc.

Prior to this commit, if only a single value was desired one would have
to use the format:

```yaml
possible_values:
    - value
```

But now once can use

```yaml
possible_values: value
```

Closes #614
Closes #613
@homu
Copy link
Contributor

homu commented Aug 20, 2016

⌛ Testing commit 9ade2cd with merge 098d7fa...

@homu
Copy link
Contributor

homu commented Aug 20, 2016

☀️ Test successful - status

@homu homu merged commit 9ade2cd into master Aug 20, 2016
@kbknapp kbknapp deleted the issue-614 branch August 21, 2016 00:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants