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

fqm-execution bump to 1.3.1 and update Docker setup #141

Closed
wants to merge 10 commits into from

Conversation

hossenlopp
Copy link
Contributor

Summary

New behavior

  • npm run check no longer results in those two lint errors from ecqm-content-r4-2021/input/images/cql.js. I added a .eslintignore file for the ecqm-content-r4-2021 directory. I hope this is okay, let me know if it's not. I was tired of seeing those and now the prettier check seems to actually happen.
  • Otherwise, there should be no new behaviors.
  • upload-bundles npm script now is able to handle a directory of nested MADiE Measures and TestCase export bundles.
    Note: these will not execute in the server at the moment due to issues with relatedArtifact references in the Measure and Library resources and patient data queries

Code changes

  • .eslintignore - added this file to avoid those errors that we (I think) don't care about
  • package.json / package-lock.json - further upgraded the fqm-execution version
  • execQueue.js - Updated to use newer MeasureReportBuilder API.

Code changes Lauren made in previous fqm-e version bump PR

  • ci.yml, measure.service.js, base.service.test.js

Testing guidance

  • npm run check Reminder that sometimes a couple of these tests will fail. That is okay for now and something we need to address in separate tasking.
  • npm start make sure that all functionality works the same.

NOTE:

  • We had an in review PR for updating the fqm-execution version that I closed and just branched off of to upgrade the version even further. The previous PR is here for reference.

@hossenlopp
Copy link
Contributor Author

This wont work either 👎

@hossenlopp hossenlopp closed this Sep 13, 2023
@github-actions
Copy link

Coverage report

Total coverage

Status Category Percentage Covered / Total
🟢 Statements 93.67% 1287/1374
🟢 Branches 86.28% 371/430
🟢 Functions 93.75% 210/224
🟢 Lines 93.58% 1254/1340

Status of coverage: 🟢 - ok, 🟡 - slightly more than threshold, 🔴 - under the threshold

Show files with reduced coverage 🔻

Reduced coverage

Status Filename Statements Branches Functions Lines
🟢 src/queue/execQueue.js 93.88% (-3.85% 🔻) 100% 90% (+1.11% 🔼) 93.48% (-4.14% 🔻)

Status of coverage: 🟢 - ok, 🟡 - slightly more than threshold, 🔴 - under the threshold

Report generated by 🧪jest coverage report action from 5472b80

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