Skip to content

Conversation

@ePaul
Copy link
Contributor

@ePaul ePaul commented Jun 17, 2017

PR checklist

Description of the PR

As discussed in #5147, all the options which previously had comma separated lists of values now also allow multiple values (or multiple key-value-pairs), as was previously (in #5191) already done for -D.
This affects those with key-value-pairs:

--additional-properties
--import-mappings
--instantiation-types
--reserved-word-mappings
--type-mappings

and this option with a simple list of values:

--language-specific-primitives

I also regenerated the output of help generate in README.md, which was already out-of-date before.

@wing328 wing328 merged commit d31d431 into swagger-api:master Jun 28, 2017
@wing328 wing328 added this to the v2.2.3 milestone Jun 28, 2017
@ePaul ePaul deleted the 5147-plural-other-properties branch June 28, 2017 11:39
@wing328 wing328 changed the title #5147: allow multiple for other plural options, too Allow multiple for other plural options (e.g. --import-mappings) Jun 28, 2017
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.

2 participants