Rel 6 1 mergeback #3927
Merged
Rel 6 1 mergeback #3927
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
* Bump for CVE * Bump spring-data version * Fix compile * Cut over to spring bom
…esponse (#3882) * Finish up changelog, minor refactor * reset buffer only * Hack for some replacements * Failure handling * wip
* Fixed the issue * Changelog modification * Changelog modification
#3709) * Implemented seventh character extended code and the corresponding display * Modifications * Changes on previous test according to modifications made in ICD10-CM XML file
* fixed bug and added test * refactor
* updated pointcuts to work as intended * added changelog * review fixes Co-authored-by: leif stawnyczy <[email protected]>
…e left orphaned (#3905) * Add test and implementation * Add changelog
* Add implementation, changelog, test * Update hapi-fhir-jpaserver-test-utilities/src/test/java/ca/uhn/fhir/jpa/provider/r4/ResourceProviderR4Test.java Co-authored-by: Ken Stevens <[email protected]> Co-authored-by: Ken Stevens <[email protected]>
* Applying all changes from previous dev branch to current one pointing to rel_6_1 * Fixing merge conflict related to Msg.code value. * Fixing Msg.code value. * Making checkstyle happy. * Making sure that all tests are passing. * Passing all tests after fixing Msg.code * Passing all tests. Co-authored-by: peartree <[email protected]>
#3746) * 3745 - fixed NPE for bundle with duplicate conditional create resources and a conditional delete * created unit test for skip of delete operation while processing duplicating create entries * moved unit test to FhirSystemDaoR4Test
* added MdmLinkCreateSvcimplTest * fixed creating mdm-link not setting the resource type correctly * fixed a bug where ResourcePersistenceId was being duplicated instead of passed on * Update hapi-fhir-jpaserver-mdm/src/test/java/ca/uhn/fhir/jpa/mdm/svc/MdmLinkCreateSvcImplTest.java Change order of tests such that assertEquals takes expected value then actual value Co-authored-by: jdar8 <[email protected]> * added changelog, also changed a setup function in test to beforeeach Co-authored-by: Long Ma <[email protected]> Co-authored-by: jdar8 <[email protected]>
* Fix to the issue * Progress * fixed the issue * Addressing suggestions * add response status code to MethodOutcome * Addressing suggestions Co-authored-by: Ken Stevens <[email protected]>
* Respect caching in bullk export, fix bug with completed date on empty jobs * add changelog * Add impl * Add breaking test * Complete failing test * more broken tests * Fix more tests' * Fix paging bug * Fix another brittle test
* do not attempt to merge compartment permissions with filters * changelog * Rename to IT for concurrency problems Co-authored-by: Tadgh <[email protected]>
Co-authored-by: Ken Stevens <[email protected]>
* begin with failing test * fixed * change log * rollback default count change and corresponding comments Co-authored-by: Ken Stevens <[email protected]>
… they don't slow down startup.
* lower batchsize of delete requests so that we do not get sql exceptions * blah * fix test * updated tests to not fail Co-authored-by: leif stawnyczy <[email protected]>
|
This pull request fixes 2 alerts when merging b8794d6 into 7f7b7d6 - view on LGTM.com fixed alerts:
|
|
This pull request fixes 2 alerts when merging dfc6b93 into 7f7b7d6 - view on LGTM.com fixed alerts:
|
|
This pull request fixes 2 alerts when merging eb7e643 into 7f7b7d6 - view on LGTM.com fixed alerts:
|
|
This pull request fixes 2 alerts when merging 11eb37b into 7f7b7d6 - view on LGTM.com fixed alerts:
|
TipzCM
approved these changes
Aug 17, 2022
|
This pull request fixes 2 alerts when merging b161cf8 into 7f7b7d6 - view on LGTM.com fixed alerts:
|
|
This pull request fixes 2 alerts when merging 8058829 into 7f7b7d6 - view on LGTM.com fixed alerts:
|
Codecov Report
@@ Coverage Diff @@
## master #3927 +/- ##
============================================
+ Coverage 81.82% 81.87% +0.05%
- Complexity 20769 20838 +69
============================================
Files 1417 1421 +4
Lines 77089 77246 +157
Branches 11061 11078 +17
============================================
+ Hits 63075 63243 +168
+ Misses 9869 9858 -11
Partials 4145 4145
Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here. |
fil512
approved these changes
Aug 18, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Bring in all release branch work back to master