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

Packaging and DRYing #148

Merged
merged 22 commits into from
Oct 24, 2023
Merged

Packaging and DRYing #148

merged 22 commits into from
Oct 24, 2023

Conversation

t-young31
Copy link
Member

@t-young31 t-young31 commented Oct 12, 2023

Reduces a lot of duplication and improves DX. Drive by fixes the system test that was missing some env vars. Moves token_buffer/patient_queue/core API functionality to a shared pixl_core package. Moves the report de-id into pixl_ehr.

@t-young31 t-young31 requested a review from stefpiatek October 12, 2023 13:43
@t-young31 t-young31 marked this pull request as ready for review October 12, 2023 13:43
stefpiatek
stefpiatek previously approved these changes Oct 24, 2023
cli/README.md Outdated Show resolved Hide resolved
hasher/README.md Outdated Show resolved Hide resolved
orthanc/orthanc-anon/plugin/pixl.py Outdated Show resolved Hide resolved
pixl_core/README.md Outdated Show resolved Hide resolved
pixl_ehr/README.md Outdated Show resolved Hide resolved
pixl_pacs/README.md Outdated Show resolved Hide resolved
@t-young31 t-young31 merged commit 909868c into main Oct 24, 2023
7 checks passed
@t-young31 t-young31 deleted the t-young31/packaging branch October 24, 2023 16:27
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