Skip to content

Conversation

@jamesbassett
Copy link

As a follow up to #192, this PR adds support for exclusiveMin and exclusiveMax by changing the min/max label, i.e.

exclusiveMinimum == false: Minimum value
exclusiveMinimum == true: Minimum value (exclusive)
exclusiveMaximum == false: Maximum value
exclusiveMaximum == true: Maximum value (exclusive)

Inclusive is assumed to be the default, so there's no (inclusive) required.

@coveralls
Copy link

Coverage Status

Coverage increased (+0.1%) to 85.013% when pulling cd6bad7 on jamesbassett:min-max-exclusive into 908f524 on Swagger2Markup:master.

@RobWin
Copy link
Member

RobWin commented Oct 14, 2016

Perfect. Thx.

@RobWin RobWin merged commit 5871f0c into Swagger2Markup:master Oct 14, 2016
@jamesbassett jamesbassett deleted the min-max-exclusive branch October 14, 2016 23:07
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.

3 participants