Skip to content

Conversation

@pwildenhain
Copy link
Collaborator

Closes #293

@pwildenhain pwildenhain self-assigned this Apr 15, 2025
@pwildenhain pwildenhain requested a review from Copilot April 15, 2025 16:48
Copy link

Copilot AI left a 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 = [

Fix tempfile creation for windows
@codecov
Copy link

codecov bot commented Apr 15, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (4bf25e5) to head (cdcafb1).
Report is 2 commits behind head on master.

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     
Files with missing lines Coverage Δ
redcap/methods/__init__.py 100.00% <100.00%> (ø)
redcap/methods/base.py 100.00% <ø> (ø)
redcap/methods/file_repository.py 100.00% <100.00%> (ø)
redcap/methods/files.py 100.00% <100.00%> (ø)
redcap/methods/logging.py 100.00% <ø> (ø)
redcap/project.py 100.00% <ø> (ø)
redcap/request.py 100.00% <100.00%> (ø)

... and 1 file with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@pwildenhain pwildenhain merged commit 01b33c9 into master Apr 15, 2025
11 checks passed
@pwildenhain pwildenhain deleted the add-file-repo-api branch April 15, 2025 17:44
@jthomas08
Copy link

Paul did this make it to 2.6.0? I didn't see file_repo API in the docs.

@pwildenhain
Copy link
Collaborator Author

Not yet, I was going to finish the survey API work in #298 before cutting a release and updating the docs

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Support File Repo Actions

3 participants