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

Update fqmexec 1.3.1 #139

Closed
wants to merge 6 commits into from
Closed

Update fqmexec 1.3.1 #139

wants to merge 6 commits into from

Conversation

elsaperelli
Copy link
Contributor

Summary

  • Updates to use the latest fqm-execution version (1.3.1). See here for the release notes.

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.

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

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

  • ci.yml, measure.service.js, execQueue.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.

@github-actions
Copy link

github-actions bot commented Sep 11, 2023

Coverage report

Total coverage

Status Category Percentage Covered / Total
🟢 Statements 93.8% 1286/1371
🟢 Branches 86.21% 369/428
🟢 Functions 93.75% 210/224
🟢 Lines 93.72% 1253/1337

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

Report generated by 🧪jest coverage report action from 90d9ba3

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