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

V7.x dev #241

Merged
merged 7 commits into from
Mar 17, 2025
Merged

V7.x dev #241

merged 7 commits into from
Mar 17, 2025

Conversation

funkyshu
Copy link
Member

No description provided.

funkyshu and others added 6 commits March 8, 2025 13:11
* Switch to AWS SDK v2 (#212)

* Switch to AWS SDK v2

* Reduce getCopyObjectInput cyclomatic complexity

* Only apply Retryer if not nil

* Switch to correct types.NotFound error

* Keys should not have leading slash

* Leave client if only setting DisableServerSideEncryption option

* Stop swallowing writer errors

* Fix readEOFSeen behavior

---------

Co-authored-by: John Judd <[email protected]>

* Awsv2 (#232)

* update dependencies

* fix lint issue

* fix testsuite

* update doc, update changelog

* add v7.x-dev to workflows

* remove python in codeql

* Update CHANGELOG.md

* update to change module path to v7

---------

Co-authored-by: Nathan Baulch <[email protected]>
* Azure az:// scheme

* fix vfssimple

* fixed tests and added README.md and CHANGELOG.md entries

---------

Co-authored-by: John Judd <[email protected]>
* update vfs interface to use Authority and depreciate Volume and ChangeDir

* update azure backend to refactor to store authority and deprecate volume and changeddir

* fix azure changedir usage

* update ftp backend to deprecate volume and changeddir

* regen mocks to include interface changes

* update sftp backend to deprecate volume and changeddir

* fix minor lint issues with ftp backend

* update mocks

* update s3 backend to deprecate volume and changeddir

* minor spacing changes for sftp backend

* update gs backend to deprecate volume and changeddir

* the rest

* fix lint issues for tests for deprecated methods

* fix changelog

* fix location.New bug where it used the old location, even for changes in rel path

* fix pr comments

* update readme to mention the breaking changes

* update readme to reference ticket
* add deprecations in interface, add functional option. Fixes #238

* update s3 backend to deprecate vfs.Retry and vfs.Options and add newfilesystem functional options

* update azure backend to deprecate vfs.Retry and vfs.Options and add newfilesystem functional options

* update ftp backend to deprecate vfs.Retry and vfs.Options and add newfilesystem functional options

* update sftp backend to deprecate vfs.Retry and vfs.Options and add newfilesystem functional options

* update mem backend to deprecate vfs.Retry and vfs.Options and add newfilesystem functional options

* update os backend to deprecate vfs.Retry and vfs.Options and add newfilesystem functional options

* update gs backend to deprecate vfs.Retry and vfs.Options and add newfilesystem functional options

* add changelog entry

* update dependencies

* revert to storage/v1.51.0

* update doc

* add generics to simplify apply opts

* removed gs use of vfs.Retry

* updated documentation and changelog

* add comments to public functions

* fix whitespace

* updated readme

* removed redundant links in readme
@c2fo-cibot c2fo-cibot bot added the size/XXL Denotes a PR that changes 1000+ lines label Mar 17, 2025
@codecov-commenter
Copy link

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

Attention: Patch coverage is 45.45455% with 786 lines in your changes missing coverage. Please review.

Project coverage is 43.38%. Comparing base (7eaeec4) to head (b90f016).

Files with missing lines Patch % Lines
backend/s3/mocks/Client.go 0.00% 572 Missing ⚠️
mocks/Location.go 0.00% 25 Missing ⚠️
backend/s3/file.go 82.50% 17 Missing and 4 partials ⚠️
backend/gs/newFileSystemOption.go 37.50% 20 Missing ⚠️
backend/azure/client.go 26.08% 15 Missing and 2 partials ⚠️
backend/gs/file.go 56.66% 9 Missing and 4 partials ⚠️
backend/ftp/location.go 69.44% 7 Missing and 4 partials ⚠️
backend/s3/fileSystem.go 69.44% 8 Missing and 3 partials ⚠️
backend/s3/options.go 59.09% 6 Missing and 3 partials ⚠️
utils/authority/authority.go 35.71% 6 Missing and 3 partials ⚠️
... and 19 more

❗ Your organization needs to install the Codecov GitHub app to enable full functionality.

Additional details and impacted files
@@             Coverage Diff             @@
##             main     #241       +/-   ##
===========================================
+ Coverage   18.59%   43.38%   +24.79%     
===========================================
  Files          61       65        +4     
  Lines       20368     8655    -11713     
===========================================
- Hits         3787     3755       -32     
+ Misses      16250     4541    -11709     
- Partials      331      359       +28     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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

@funkyshu funkyshu merged commit b4bd716 into main Mar 17, 2025
11 checks passed
@funkyshu funkyshu deleted the v7.x-dev branch March 17, 2025 17:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
size/XXL Denotes a PR that changes 1000+ lines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants