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

src/dest mixed up in AzClient.upload_data_to_remote_location? #2

Open
embario opened this issue May 1, 2024 · 0 comments
Open

src/dest mixed up in AzClient.upload_data_to_remote_location? #2

embario opened this issue May 1, 2024 · 0 comments

Comments

@embario
Copy link

embario commented May 1, 2024

The AzClient method:

    def upload_data_to_remote_location(
        self,
        src: AzRemoteSASLocation,
        dest: AzLocalLocation,
        transfer_options: AzCopyOptions,
    ) -> AzCopyJobInfo:
        return self._copy(src=src, dest=dest, transfer_options=transfer_options)

should have AzLocalLocation as its src type and AzRemoteSASLocation as its dest type. Not a big deal but something I noticed. FWIW I tried to make a PR but repo's pretty restrictive. Open to iterate updates?

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

No branches or pull requests

1 participant