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

Enable Writes to Azure 2 #134

Closed
wants to merge 30 commits into from
Closed

Conversation

mightyshazam
Copy link
Collaborator

This PR is an extension of #114 by @thovoll . It fixes the broken tests, adds support for ARM and adds a test for Azure Blob Storage using Azurite.

@houqp houqp requested review from xianwill and mosyp March 20, 2023 01:30
@houqp houqp requested a review from thovoll March 20, 2023 01:30
@rtyler rtyler self-assigned this Mar 24, 2023
@rtyler
Copy link
Member

rtyler commented Mar 24, 2023

Thanks for the work! Give me some time to review it and compare with the work I had been doing in a branch as well 🤗

@rtyler
Copy link
Member

rtyler commented Mar 25, 2023

The cargo clippy errors are worth addressing, there are some around futures not being resolved which is definitely a bad runtime behavior. Running the build and tests locally is looking generally good. @mightyshazam do you mind if I pull this into a topic branch on this repository, and then we can collaborate on cleaning it up for merge and release?

@mightyshazam
Copy link
Collaborator Author

The cargo clippy errors are worth addressing, there are some around futures not being resolved which is definitely a bad runtime behavior. Running the build and tests locally is looking generally good. @mightyshazam do you mind if I pull this into a topic branch on this repository, and then we can collaborate on cleaning it up for merge and release?

Sounds good to me.

@rtyler
Copy link
Member

rtyler commented Mar 26, 2023

@mightyshazam I have pushed the azure-writes branch and sent you an invite to act as a collaborator on this repo for that branch. I will close this pull request in favor of the draft: #136

@rtyler rtyler closed this Mar 26, 2023
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.

4 participants