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

Prevent chunk from returning to in-progress unless it is errorred, in… #4417

Merged
merged 5 commits into from
Jan 10, 2023

Conversation

tadgh
Copy link
Collaborator

@tadgh tadgh commented Jan 9, 2023

…-progress, or queued

@codecov
Copy link

codecov bot commented Jan 9, 2023

Codecov Report

Base: 81.00% // Head: 81.29% // Increases project coverage by +0.28% 🎉

Coverage data is based on head (9af79cc) compared to base (02944ba).
Patch has no changes to coverable lines.

❗ Current head 9af79cc differs from pull request most recent head d0ba6a0. Consider uploading reports for the commit d0ba6a0 to get more accurate results

Additional details and impacted files
@@             Coverage Diff              @@
##             master    #4417      +/-   ##
============================================
+ Coverage     81.00%   81.29%   +0.28%     
- Complexity    23466    23616     +150     
============================================
  Files          1446     1421      -25     
  Lines         86175    86422     +247     
  Branches      11798    11701      -97     
============================================
+ Hits          69809    70256     +447     
+ Misses        11087    10972     -115     
+ Partials       5279     5194      -85     
Impacted Files Coverage Δ
...validation/validator/VersionTypeAdvisorDstu21.java 0.00% <0.00%> (-100.00%) ⬇️
...ir/jpa/provider/r4/BaseResourceProviderR4Test.java 0.00% <0.00%> (-95.32%) ⬇️
...ir/jpa/api/dao/MetadataKeyCurrentlyReindexing.java 40.00% <0.00%> (-60.00%) ⬇️
ca/uhn/fhir/mdm/api/IMdmLink.java 42.85% <0.00%> (-28.58%) ⬇️
...uhn/fhir/jpa/term/config/TermCodeSystemConfig.java 75.00% <0.00%> (-25.00%) ⬇️
...a/mdm/svc/candidate/FindCandidateByExampleSvc.java 66.66% <0.00%> (-22.23%) ⬇️
...a/uhn/fhir/jpa/mdm/svc/MdmLinkQuerySvcImplSvc.java 66.66% <0.00%> (-20.84%) ⬇️
...bscription/channel/impl/LinkedBlockingChannel.java 72.72% <0.00%> (-17.28%) ⬇️
...model/entity/ResourceIndexedComboStringUnique.java 56.75% <0.00%> (-16.22%) ⬇️
...hir/rest/server/servlet/ServletRequestDetails.java 73.56% <0.00%> (-15.84%) ⬇️
... and 313 more

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@tadgh tadgh enabled auto-merge (squash) January 10, 2023 02:52
@tadgh tadgh merged commit 65bf8d4 into master Jan 10, 2023
@tadgh tadgh deleted the double-delivery-woes branch January 10, 2023 03:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants