Skip to content

Non-functional change strictly to improve code quality and readability.#1597

Merged
JPercival merged 1 commit intomasterfrom
ld-20250813-clarify-pop-evaluated-resource-stack-code
Aug 15, 2025
Merged

Non-functional change strictly to improve code quality and readability.#1597
JPercival merged 1 commit intomasterfrom
ld-20250813-clarify-pop-evaluated-resource-stack-code

Conversation

@lukedegruchy
Copy link
Copy Markdown
Contributor

  • Non-functional change strictly to improve code quality and readability for State#popEvaluatedResourceStack, since the intent of the current code is difficult to ascertain but is technically correct

@github-actions
Copy link
Copy Markdown

Formatting check succeeded!

@codecov
Copy link
Copy Markdown

codecov bot commented Aug 13, 2025

Codecov Report

❌ Patch coverage is 71.42857% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 65.21%. Comparing base (4e67821) to head (9283e5e).
⚠️ Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
...va/org/opencds/cqf/cql/engine/execution/State.java 71.42% 1 Missing and 1 partial ⚠️
Additional details and impacted files
@@             Coverage Diff              @@
##             master    #1597      +/-   ##
============================================
- Coverage     65.22%   65.21%   -0.01%     
+ Complexity     1977     1976       -1     
============================================
  Files           503      503              
  Lines         29065    29069       +4     
  Branches       5689     5690       +1     
============================================
+ Hits          18957    18958       +1     
- Misses         7784     7786       +2     
- Partials       2324     2325       +1     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@sonarqubecloud
Copy link
Copy Markdown

Copy link
Copy Markdown
Contributor

@Chris0296 Chris0296 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM! Certainly more readable.

@JPercival JPercival merged commit 9761585 into master Aug 15, 2025
7 checks passed
@JPercival JPercival deleted the ld-20250813-clarify-pop-evaluated-resource-stack-code branch August 15, 2025 15:03
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.

4 participants