-
Notifications
You must be signed in to change notification settings - Fork 1.3k
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
Rel 6 1 mergeback #3927
Rel 6 1 mergeback #3927
Conversation
* 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:
|
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. |
Bring in all release branch work back to master