-
-
Notifications
You must be signed in to change notification settings - Fork 86
Add file repo api #297
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
Add file repo api #297
Conversation
Previously exporting arms when the weren't assigned to events would throw an error
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Copilot reviewed 16 out of 17 changed files in this pull request and generated 1 comment.
Files not reviewed (1)
- pytest.ini: Language not supported
Comments suppressed due to low confidence (1)
redcap/methods/file_repository.py:157
- [nitpick] The variable 'key_values' is reused in the list comprehension, which may lead to confusion. Consider renaming the inner variable (e.g., to 'pair') to improve clarity and avoid shadowing.
key_values = [
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #297 +/- ##
=========================================
Coverage 100.00% 100.00%
=========================================
Files 21 22 +1
Lines 652 684 +32
=========================================
+ Hits 652 684 +32
🚀 New features to boost your workflow:
|
|
Paul did this make it to 2.6.0? I didn't see file_repo API in the docs. |
|
Not yet, I was going to finish the survey API work in #298 before cutting a release and updating the docs |
Closes #293