Skip to content

Fix size_only arg in download_s3_path not being passed fully#28

Merged
njmei merged 1 commit intomainfrom
fix-data-sync-issue
Apr 2, 2025
Merged

Fix size_only arg in download_s3_path not being passed fully#28
njmei merged 1 commit intomainfrom
fix-data-sync-issue

Conversation

@njmei
Copy link
Collaborator

@njmei njmei commented Apr 2, 2025

No description provided.

@njmei njmei requested a review from rpmcginty April 2, 2025 18:14
@codecov
Copy link

codecov bot commented Apr 2, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 88.74%. Comparing base (97bf307) to head (42a5730).
Report is 2 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main      #28   +/-   ##
=======================================
  Coverage   88.74%   88.74%           
=======================================
  Files          37       37           
  Lines        3385     3385           
  Branches      491      491           
=======================================
  Hits         3004     3004           
  Misses        285      285           
  Partials       96       96           
Files with missing lines Coverage Δ
src/aibs_informatics_aws_utils/s3.py 91.26% <ø> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@njmei njmei force-pushed the fix-data-sync-issue branch from c105f59 to 0521b1d Compare April 2, 2025 18:18
@njmei njmei force-pushed the fix-data-sync-issue branch from 0521b1d to 42a5730 Compare April 2, 2025 18:19
@njmei njmei changed the title Fix size_only missing arg, causing should_sync unexpected behavior Fix size_only arg in download_s3_path not being passed fully Apr 2, 2025
@njmei njmei merged commit 104989e into main Apr 2, 2025
8 checks passed
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