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

Implement patient programs #92

Merged
merged 26 commits into from
Feb 28, 2024

Conversation

samamorgan
Copy link
Collaborator

@samamorgan samamorgan commented Oct 31, 2023

Resolves #73
Resolves #91

@gone The coverage comment action can't run because this is coming from a fork. Once merged, this should work as expected. Do note that the actual tests do pass, it's just the comment action lacking permissions that creates a failure state. Check the passing test here: https://github.com/Lightmatter/welkin-health/actions/runs/6749077327/job/18348727420?pr=92#step:8:49

Changelog

  • Adds dynamic patient fixture (re Tests should be stateless and repeatable #76)
  • Adds Program and Programs for instance and formation endpoints
  • Adds more capability to Patient.get
  • Omit test suite in coverage report
  • Adds pre-commit
  • Adds autoupdate action for pre-commit
  • Adds pytest coverage comment

@samamorgan samamorgan changed the title ZST-1157 Implement patient programs Implement patient programs Oct 31, 2023
@samamorgan samamorgan requested a review from gone October 31, 2023 15:05
@samamorgan samamorgan closed this Nov 3, 2023
@samamorgan samamorgan reopened this Nov 3, 2023
@samamorgan samamorgan closed this Jan 8, 2024
@samamorgan samamorgan deleted the juliand/zst-1157 branch January 8, 2024 19:37
@samamorgan samamorgan restored the juliand/zst-1157 branch January 19, 2024 22:15
@samamorgan samamorgan reopened this Jan 19, 2024
@samamorgan samamorgan merged commit 69d0296 into Lightmatter:develop Feb 28, 2024
7 of 8 checks passed
@samamorgan samamorgan deleted the juliand/zst-1157 branch February 29, 2024 16:35
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