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

XSD: Support <value> in <variable> #3478

Merged
merged 3 commits into from Jun 13, 2019
Merged

Conversation

304NotModified
Copy link
Member

@304NotModified 304NotModified commented Jun 12, 2019

as alternative to variable attribute

resolves #1577

@304NotModified 304NotModified added this to the 4.6.5 milestone Jun 12, 2019
@304NotModified 304NotModified changed the base branch from master to release/4.6.5 June 12, 2019 22:52
@304NotModified
Copy link
Member Author

304NotModified commented Jun 12, 2019

unfortunate I cannot add a validation rule, "attribute value is needed when body is null". We need XSD 1.1 for that, which isn't supported in VS 2017 (and think also not VS 2019 - seen in my tests)

(need for xs:assert)

@codecov-io
Copy link

Codecov Report

❗ No coverage uploaded for pull request base (release/4.6.5@a238637). Click here to learn what that means.
The diff coverage is n/a.

@@               Coverage Diff               @@
##             release/4.6.5   #3478   +/-   ##
===============================================
  Coverage                 ?     80%           
===============================================
  Files                    ?     361           
  Lines                    ?   28686           
  Branches                 ?    3829           
===============================================
  Hits                     ?   22957           
  Misses                   ?    4617           
  Partials                 ?    1112

@304NotModified 304NotModified merged commit 75afe1a into release/4.6.5 Jun 13, 2019
@repo-ranger repo-ranger bot deleted the xsd-value-in-variable branch June 13, 2019 00:36
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.

None yet

2 participants