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.

Sign up for GitHub

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

Jump to bottom

Source azure blob storage: certification #37504

Merged
merged 20 commits into from
Apr 29, 2024
Merged

Source azure blob storage: certification #37504

merged 20 commits into from
Apr 29, 2024

Conversation

artem1205
Copy link
Collaborator

@artem1205 artem1205 commented Apr 23, 2024 •
edited
Loading

What

Resolving https://github.com/airbytehq/airbyte-internal-issues/issues/6831

How

  • reorder spec properties
  • update docs

Review guide

  1. *.py

User Impact

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

@artem1205
Source Azure Blob Storage: add unit tests
c400532 
[skip ci]

Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: ref
42af8a4 
[skip ci]

Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: add test for stream reader
842abcb 
[skip ci]

Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: ref
bdcfeb5 
[skip ci]

Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: add freezegun && update deps
3e0fe50 
[skip ci]

Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: add tests
20e2eb9 
[skip ci]

Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: ref legacy config migration
5b966ff 
[skip ci]

Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: ref legacy config migration
956a4d6 
[skip ci]

Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: format
58a78fb 
[skip ci]

Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: bump version
04df48b 
Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: fix test
c355125 
Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: reorder spec
7f45551 
[skip ci]

Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205 artem1205 self-assigned this Apr 23, 2024
@vercel Vercel
Copy link

vercel bot commented Apr 23, 2024 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for Git ↗︎

1 Ignored Deployment
Name Status Preview Comments Updated (UTC)
airbyte-docs ⬜️ Ignored (Inspect) Visit Preview Apr 29, 2024 8:46am

@octavia-squidington-iii octavia-squidington-iii added area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/azure-blob-storage labels Apr 23, 2024
@artem1205
Source Azure Blob Storage: reorder spec
3515425 
[skip ci]

Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: Add Prerequisites
6894bc6 
[skip ci]

Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: update metadata
4a69e4e 
[skip ci]

Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: update docs
5f859e0 
[skip ci]

Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: add error handler
c0789cf 
[skip ci]

Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205 artem1205 requested a review from a team April 23, 2024 17:31
@artem1205
Merge remote-tracking branch 'refs/remotes/origin/master' into artem1…
24ac101 
…205/source-azure-blob-storage-cert-docs

# Conflicts:
#	airbyte-integrations/connectors/source-azure-blob-storage/metadata.yaml
#	airbyte-integrations/connectors/source-azure-blob-storage/pyproject.toml
#	airbyte-integrations/connectors/source-azure-blob-storage/source_azure_blob_storage/stream_reader.py
#	docs/integrations/sources/azure-blob-storage.md
@artem1205
Source Azure Blob Storage: add unit test
643d774 
Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205
Source Azure Blob Storage: set maxSecondsBetweenMessages
5a929a9 
Signed-off-by: Artem Inzhyyants <artem.inzhyyants@gmail.com>
@artem1205 artem1205 marked this pull request as ready for review April 29, 2024 08:48
@octavia-squidington-iv octavia-squidington-iv requested a review from a team April 29, 2024 08:50
@artem1205 artem1205 merged commit 8d6f159 into master Apr 29, 2024
38 of 39 checks passed
@artem1205 artem1205 deleted the artem1205/source-azure-blob-storage-cert-docs branch April 29, 2024 12:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@tolik0 tolik0 tolik0 approved these changes

Assignees

@artem1205 artem1205

Labels
area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/azure-blob-storage
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@artem1205 @tolik0 @octavia-squidington-iii