Pull requests into cql-execution require the following.
Submitter and reviewer should ✔ when done.
For items that are not-applicable, mark "N/A" and ✔.
CDS Connect and Bonnie are the main users of this repository.
It is strongly recommended to include a person from each of those projects as a reviewer.
Submitter:
[x] This pull request describes why these changes were made
[x] Code diff has been done and been reviewed (it does not contain: additional white space, not applicable code changes, debug statements, etc.)
[x] Tests are included and test edge cases
[x] Tests have been run locally and pass
[x] Code coverage has not gone down and all code touched or added is covered.
[x] Code passes lint and prettier (hint: use yarn run test:plus to run tests, lint, and prettier)
[N/A] All dependent libraries are appropriately updated or have a corresponding PR related to this change
[x] cql4browsers.js built with yarn run build:browserify if source changed.
Reviewer:
Name:
[x] Code is maintainable and reusable, reuses existing code and infrastructure where appropriate, and accomplishes the task’s purpose
[x] The tests appropriately test the new code, including edge cases
Implementation of the cql 1.5 aggregate query functionality described here in the spec :https://cql.hl7.org/2020May/04-logicalspecification.html#aggregateclause
Pull requests into cql-execution require the following. Submitter and reviewer should ✔ when done. For items that are not-applicable, mark "N/A" and ✔.
CDS Connect and Bonnie are the main users of this repository. It is strongly recommended to include a person from each of those projects as a reviewer.
Submitter:
yarn run test:plusto run tests, lint, and prettier)cql4browsers.jsbuilt withyarn run build:browserifyif source changed.Reviewer:
Name: