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

Allow patches to be authorized for DSTU3 transactions #1529

Merged
merged 3 commits into from Oct 8, 2019

Conversation

@jamesagnew
Copy link
Owner

jamesagnew commented Oct 8, 2019

No description provided.

@jamesagnew jamesagnew requested a review from fil512 Oct 8, 2019
@fil512
fil512 approved these changes Oct 8, 2019
@codecov

This comment has been minimized.

Copy link

codecov bot commented Oct 8, 2019

Codecov Report

Merging #1529 into master will increase coverage by 0.04%.
The diff coverage is 60%.

Impacted file tree graph

@@             Coverage Diff              @@
##             master    #1529      +/-   ##
============================================
+ Coverage     74.69%   74.74%   +0.04%     
- Complexity    12462    12478      +16     
============================================
  Files           904      904              
  Lines         51691    51705      +14     
  Branches       8677     8681       +4     
============================================
+ Hits          38612    38647      +35     
+ Misses         9588     9576      -12     
+ Partials       3491     3482       -9
Impacted Files Coverage Δ Complexity Δ
.../fhir/rest/server/interceptor/auth/RuleImplOp.java 76.61% <37.5%> (-0.75%) 108 <0> (+2)
...ase/src/main/java/ca/uhn/fhir/util/BundleUtil.java 92.7% <75%> (-0.33%) 32 <1> (+2)
...dstu3/TransactionProcessorVersionAdapterDstu3.java 87.03% <75%> (+1.32%) 24 <0> (ø) ⬇️
...base/src/main/java/ca/uhn/fhir/util/StopWatch.java 83.09% <0%> (-3.53%) 40% <0%> (-1%)
...-base/src/main/java/ca/uhn/fhir/model/api/Tag.java 54.05% <0%> (-2.71%) 20% <0%> (-1%)
...uhn/fhir/rest/server/method/ReadMethodBinding.java 73.41% <0%> (-2.54%) 26% <0%> (-1%)
...n/fhir/jpa/config/HapiFhirHibernateJpaDialect.java 66.66% <0%> (ø) 9% <0%> (+1%) ⬆️
.../java/ca/uhn/fhir/rest/client/impl/BaseClient.java 84.49% <0%> (+0.38%) 67% <0%> (+1%) ⬆️
...rc/main/java/ca/uhn/fhir/context/ModelScanner.java 77.73% <0%> (+0.7%) 75% <0%> (+1%) ⬆️
... and 8 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 1f1b94a...0d31a56. Read the comment docs.

@jamesagnew jamesagnew merged commit 861ed36 into master Oct 8, 2019
3 of 5 checks passed
3 of 5 checks passed
LGTM analysis: Java Analysis failed (could not build the base commit (1f1b94a))
Details
codecov/patch 60% of diff hit (target 74.69%)
Details
LGTM analysis: JavaScript No code changes detected
Details
codecov/project 74.74% (+0.04%) compared to 1f1b94a
Details
jamesagnew.hapi-fhir Build #20191008.12 succeeded
Details
@jamesagnew jamesagnew deleted the ja_20190810_auth_patch_for_transaction branch Oct 8, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
2 participants
You can’t perform that action at this time.