-
Notifications
You must be signed in to change notification settings - Fork 27
feat: Clarify FhirData setter behavior with explicit replace parameter and transition to uv #149
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
Conversation
|
@jenniferjiangkells The issue with the slow CI testing for 3.9 python has been confirmed to be the result of spacy/thinc removing pre-built wheels for Python 3.9. It is still not clear why the same issue didn't arise for poetry/pip installation and only uv. It is possible that these wheels are cached across jobs and so still available to the CI run. Either way I have removed support for python 3.9 as it is EOL. |
|
Thanks! @adamkells and @jenniferjiangkells |
Description
Clarifies the behavior of resource list setters in the FhirData class by adding explicit control over whether they add to or replace existing resources.
Also transition from poetry to uv as dependency manager.
Related Issue
Closes #123 - Clarify Resource List Setter Behavior in FhirData
Changes Made
Testing
Checklist
Additional Notes
The implementation uses a dict-based approach for the replace parameter since Python property setters can only accept a single value argument. This provides explicit control while maintaining full backward compatibility with existing code.