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

CQL Command Expression Evaluation #78

Merged
merged 5 commits into from
Aug 10, 2024

Conversation

jreyno77
Copy link
Contributor

@jreyno77 jreyno77 commented Aug 9, 2024

Wire up expression to the actual engine evaluation

Wire up expression to the actual engine evaluation
Copy link

github-actions bot commented Aug 9, 2024

Formatting check succeeded!

@holly-smile holly-smile changed the base branch from feature-remote-execution to remote-url August 10, 2024 08:05
Copy link

codecov bot commented Aug 10, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 44.48%. Comparing base (48fe0d1) to head (7fdb679).

Additional details and impacted files
@@              Coverage Diff              @@
##             remote-url      #78   +/-   ##
=============================================
  Coverage         44.48%   44.48%           
  Complexity          157      157           
=============================================
  Files                39       39           
  Lines              1016     1016           
  Branches            111      111           
=============================================
  Hits                452      452           
  Misses              512      512           
  Partials             52       52           

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

@holly-smile holly-smile merged commit 5e8cafd into remote-url Aug 10, 2024
5 of 6 checks passed
@holly-smile holly-smile deleted the cql-command-expression-evaluation branch August 10, 2024 08:13
JPercival pushed a commit that referenced this pull request Aug 15, 2024
* Support remote repositories

* Add DebugCql contribution test

* Normalize path for Windows; fix assertion not running due to unawaited future

* More tests, hopefully more coverage

* Remove remote test

* Run spotless

* CQL Command Expression Evaluation (#78)

* WIP on master

Wire up expression to the actual engine evaluation

* Run spotless

* Remove irrelevant tests

* Restore test

---------

Co-authored-by: Holly Wu <holly.wu@smiledigitalhealth.com>

* Replace backslashes with forward slashes in Windows paths

* Forgot a return path

---------

Co-authored-by: Joshua Reynolds <48231325+jreyno77@users.noreply.github.com>
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.

3 participants