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

Added R4 to the client's server version check. Also tweaked the ClientServerValidation* tests to be a little more predictable #711

Merged
merged 1 commit into from Oct 26, 2017

Conversation

Projects
None yet
3 participants
@InfiniteLoop90
Contributor

InfiniteLoop90 commented Aug 13, 2017

Added R4 to the client's server version check. Also tweaked the ClientServerValidation* tests to be a little more predictable.

Added R4 to the client's server version check. Also tweaked the Clien…
…tServerValidation* tests to be a little more predictable
@InfiniteLoop90

This comment has been minimized.

Show comment
Hide comment
@InfiniteLoop90

InfiniteLoop90 Aug 13, 2017

Contributor

This is related to issue #706. Now that the R4 structures are merged into master, added R4 to the client's server FHIR version check.

Contributor

InfiniteLoop90 commented Aug 13, 2017

This is related to issue #706. Now that the R4 structures are merged into master, added R4 to the client's server FHIR version check.

@coveralls

This comment has been minimized.

Show comment
Hide comment
@coveralls

coveralls Aug 13, 2017

Coverage Status

Coverage increased (+0.01%) to 80.002% when pulling 9a6f4f8 on InfiniteLoop90:master into 04f1629 on jamesagnew:master.

coveralls commented Aug 13, 2017

Coverage Status

Coverage increased (+0.01%) to 80.002% when pulling 9a6f4f8 on InfiniteLoop90:master into 04f1629 on jamesagnew:master.

jamesagnew added a commit that referenced this pull request Oct 26, 2017

@jamesagnew

This comment has been minimized.

Show comment
Hide comment
@jamesagnew

jamesagnew Oct 26, 2017

Owner

Looks great, thanks for the contribution! Merging now.

Owner

jamesagnew commented Oct 26, 2017

Looks great, thanks for the contribution! Merging now.

@jamesagnew jamesagnew merged commit 0875318 into jamesagnew:master Oct 26, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage increased (+0.01%) to 80.002%
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment