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

✨ [greenhouse] [iterable] [linkedin-ads] [paypal-transactions] [pinterest] Bump cdk versions for to use continue on stream per-error reporting #35465

Merged
merged 7 commits into from
Feb 22, 2024
Merged

✨ [greenhouse] [iterable] [linkedin-ads] [paypal-transactions] [pinterest] Bump cdk versions for to use continue on stream per-error reporting #35465

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
7 commits
Select commit Hold shift + click to select a range
8db9398
bump cdk versions for greenhouse, iterable, linkedin-ads, paypap-tran…
brianjlai Feb 21, 2024
d8a2e25
update changelogs w/ PR id
brianjlai Feb 21, 2024
bee93ed
update lock files
brianjlai Feb 21, 2024
d57cca6
fix greenhouse unit test w/ cdk upgrade
brianjlai Feb 21, 2024
8be255b
enable pypi publishing for source-linkedin-ads
brianjlai Feb 21, 2024
ee1c237
pinterest expected record -_-
brianjlai Feb 21, 2024
2186278
add code coverage ignore for run.py for a few sources
brianjlai Feb 22, 2024
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
3 changes: 3 additions & 0 deletions airbyte-integrations/connectors/source-greenhouse/.coveragerc
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
[run]
omit =
source_greenhouse/run.py
2 changes: 1 addition & 1 deletion airbyte-integrations/connectors/source-greenhouse/metadata.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ data:
connectorSubtype: api
connectorType: source
definitionId: 59f1e50a-331f-4f09-b3e8-2e8d4d355f44
dockerImageTag: 0.4.5
dockerImageTag: 0.5.0
dockerRepository: airbyte/source-greenhouse
documentationUrl: https://docs.airbyte.com/integrations/sources/greenhouse
githubIssueLabel: source-greenhouse
Expand Down
183 changes: 59 additions & 124 deletions airbyte-integrations/connectors/source-greenhouse/poetry.lock
View file
Open in desktop

Large diffs are not rendered by default.

4 changes: 2 additions & 2 deletions airbyte-integrations/connectors/source-greenhouse/pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ requires = [ "poetry-core>=1.0.0",]
build-backend = "poetry.core.masonry.api"

[tool.poetry]
version = "0.4.5"
version = "0.5.0"
name = "source-greenhouse"
description = "Source implementation for Greenhouse."
authors = [ "Airbyte <[email protected]>",]
Expand All @@ -17,7 +17,7 @@ include = "source_greenhouse"

[tool.poetry.dependencies]
python = "^3.9,<3.12"
airbyte-cdk = "==0.55.2"
airbyte-cdk = "==0.63.2"
dataclasses-jsonschema = "==2.15.1"

[tool.poetry.scripts]
Expand Down
6 changes: 3 additions & 3 deletions airbyte-integrations/connectors/source-greenhouse/unit_tests/test_streams.py
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -138,7 +138,7 @@ def test_parse_response_expected_response(applications_stream):
]
"""
response._content = response_content
parsed_response = applications_stream.retriever._parse_response(response, stream_state={})
parsed_response = applications_stream.retriever._parse_response(response, stream_state={}, records_schema={})
records = [dict(record) for record in parsed_response]

assert records == json.loads(response_content)
Expand All @@ -148,7 +148,7 @@ def test_parse_response_empty_content(applications_stream):
response = requests.Response()
response.status_code = 200
response._content = b"[]"
parsed_response = applications_stream.retriever._parse_response(response, stream_state={})
parsed_response = applications_stream.retriever._parse_response(response, stream_state={}, records_schema={})
records = [record for record in parsed_response]

assert records == []
Expand All @@ -164,7 +164,7 @@ def test_ignore_403(applications_stream):
response = requests.Response()
response.status_code = 403
response._content = b""
parsed_response = applications_stream.retriever._parse_response(response, stream_state={})
parsed_response = applications_stream.retriever._parse_response(response, stream_state={}, records_schema={})
records = [record for record in parsed_response]
assert records == []

Expand Down
3 changes: 3 additions & 0 deletions airbyte-integrations/connectors/source-iterable/.coveragerc
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -0,0 +1,3 @@
[run]
omit =
source_iterable/run.py
2 changes: 1 addition & 1 deletion airbyte-integrations/connectors/source-iterable/metadata.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,7 @@ data:
connectorSubtype: api
connectorType: source
definitionId: 2e875208-0c0b-4ee4-9e92-1cb3156ea799
dockerImageTag: 0.2.2
dockerImageTag: 0.3.0
dockerRepository: airbyte/source-iterable
documentationUrl: https://docs.airbyte.com/integrations/sources/iterable
githubIssueLabel: source-iterable
Expand Down
28 changes: 14 additions & 14 deletions airbyte-integrations/connectors/source-iterable/poetry.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions airbyte-integrations/connectors/source-iterable/pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ requires = [ "poetry-core>=1.0.0",]
build-backend = "poetry.core.masonry.api"

[tool.poetry]
version = "0.2.2"
version = "0.3.0"
name = "source-iterable"
description = "Source implementation for Iterable."
authors = [ "Airbyte <[email protected]>",]
Expand All @@ -18,7 +18,7 @@ include = "source_iterable"
[tool.poetry.dependencies]
python = "^3.9,<3.12"
pendulum = "==2.1.2"
airbyte-cdk = "==0.58.8"
airbyte-cdk = "==0.63.2"
requests = "==2.31.0"
python-dateutil = "==2.8.2"

Expand Down
5 changes: 2 additions & 3 deletions airbyte-integrations/connectors/source-linkedin-ads/metadata.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ data:
connectorSubtype: api
connectorType: source
definitionId: 137ece28-5434-455c-8f34-69dc3782f451
dockerImageTag: 0.6.8
dockerImageTag: 0.7.0
dockerRepository: airbyte/source-linkedin-ads
documentationUrl: https://docs.airbyte.com/integrations/sources/linkedin-ads
githubIssueLabel: source-linkedin-ads
Expand All @@ -21,8 +21,7 @@ data:
name: LinkedIn Ads
remoteRegistries:
pypi:
enabled: false
# TODO: Set enabled=true after `airbyte-lib-validate-source` is passing.
enabled: true
packageName: airbyte-source-linkedin-ads
registries:
cloud:
Expand Down
36 changes: 18 additions & 18 deletions airbyte-integrations/connectors/source-linkedin-ads/poetry.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

4 changes: 2 additions & 2 deletions airbyte-integrations/connectors/source-linkedin-ads/pyproject.toml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -3,7 +3,7 @@ requires = [ "poetry-core>=1.0.0",]
build-backend = "poetry.core.masonry.api"

[tool.poetry]
version = "0.6.8"
version = "0.7.0"
name = "source-linkedin-ads"
description = "Source implementation for Linkedin Ads."
authors = [ "Airbyte <[email protected]>",]
Expand All @@ -17,7 +17,7 @@ include = "source_linkedin_ads"

[tool.poetry.dependencies]
python = "^3.9,<3.12"
airbyte-cdk = "==0.58.8"
airbyte-cdk = "==0.63.2"

[tool.poetry.scripts]
source-linkedin-ads = "source_linkedin_ads.run:run"
Expand Down
2 changes: 1 addition & 1 deletion airbyte-integrations/connectors/source-paypal-transaction/metadata.yaml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -11,7 +11,7 @@ data:
connectorSubtype: api
connectorType: source
definitionId: d913b0f2-cc51-4e55-a44c-8ba1697b9239
dockerImageTag: 2.3.0
dockerImageTag: 2.4.0
dockerRepository: airbyte/source-paypal-transaction
documentationUrl: https://docs.airbyte.com/integrations/sources/paypal-transaction
githubIssueLabel: source-paypal-transaction
Expand Down
Loading
Loading