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 Klaviyo: bump CDK dependency #17422

Merged
merged 3 commits into from
Sep 30, 2022
Merged

Source Klaviyo: bump CDK dependency #17422

merged 3 commits into from
Sep 30, 2022

Conversation

pedroslopez
Copy link
Contributor

What

Republish kalivyo to bump CDK dependency and bring in #17367, unblocking investigation for https://github.com/airbytehq/oncall/issues/568

@github-actions github-actions bot added area/connectors Connector related issues area/documentation Improvements or additions to documentation labels Sep 29, 2022
@pedroslopez
Copy link
Contributor Author

pedroslopez commented Sep 29, 2022 •
edited by github-actions bot
Loading

/test connector=connectors/source-klaviyo

🕑 connectors/source-klaviyo https://github.com/airbytehq/airbyte/actions/runs/3155175666
✅ connectors/source-klaviyo https://github.com/airbytehq/airbyte/actions/runs/3155175666
Python tests coverage:

Name                         Stmts   Miss  Cover
------------------------------------------------
source_klaviyo/__init__.py       2      0   100%
source_klaviyo/streams.py      136     10    93%
source_klaviyo/source.py        16      3    81%
------------------------------------------------
TOTAL                          154     13    92%
	 Name                                                 Stmts   Miss  Cover   Missing
	 ----------------------------------------------------------------------------------
	 source_acceptance_test/base.py                          10      4    60%   15-18
	 source_acceptance_test/config.py                        83      6    93%   78-80, 84-86
	 source_acceptance_test/conftest.py                     164    164     0%   6-282
	 source_acceptance_test/plugin.py                        48     48     0%   6-104
	 source_acceptance_test/tests/test_core.py              329    111    66%   39, 50-58, 63-70, 74-75, 79-80, 164, 202-219, 228-236, 240-245, 251, 284-289, 327-334, 374-376, 379, 439-448, 477-478, 484, 487, 520-530, 543-568, 573-577
	 source_acceptance_test/tests/test_full_refresh.py       52      2    96%   34, 65
	 source_acceptance_test/tests/test_incremental.py       152     26    83%   21-23, 29-31, 36-43, 48-61, 239, 250-258
	 source_acceptance_test/utils/asserts.py                 37      2    95%   57-58
	 source_acceptance_test/utils/common.py                  77     17    78%   15-16, 24-30, 47-54, 64, 67
	 source_acceptance_test/utils/compare.py                 62     23    63%   21-51, 68, 97-99
	 source_acceptance_test/utils/connector_runner.py       112     50    55%   23-26, 32, 36, 39-67, 70-72, 75-77, 80-82, 85-87, 90-92, 95-113, 147-149
	 source_acceptance_test/utils/json_schema_helper.py     105     13    88%   30-31, 38, 41, 65-68, 96, 120, 190-192
	 ----------------------------------------------------------------------------------
	 TOTAL                                                 1358    466    66%

Build Passed

Test summary info:

All Passed

@pedroslopez
Copy link
Contributor Author

pedroslopez commented Sep 30, 2022 •
edited by github-actions bot
Loading

/publish connector=connectors/source-klaviyo

🕑 Publishing the following connectors:
connectors/source-klaviyo
https://github.com/airbytehq/airbyte/actions/runs/3155880141


Connector Did it publish? Were definitions generated?
connectors/source-klaviyo ✅ ✅

if you have connectors that successfully published but failed definition generation, follow step 4 here ▶️

@pedroslopez pedroslopez merged commit 1dc1d54 into master Sep 30, 2022
@pedroslopez pedroslopez deleted the pedroslopez/klaviyo-bump-0.90 branch September 30, 2022 14:32
artem1205 pushed a commit that referenced this pull request Sep 30, 2022
@pedroslopez @artem1205
Source Klaviyo: bump CDK dependency (#17422)
f1168da
letiescanciano added a commit that referenced this pull request Oct 3, 2022
@letiescanciano
Merge branch 'master' into leti/add-segment-track-to-suggested-destin…
e6264e6 
…ations

* master: (75 commits)
  source-sentry: migrate to per-stream states (#17466)
  Greg/clickhouse polishing (#17483)
  upgrade debezium version to 1.9.6 (#17459)
  🐛 Source Twilio: Lookback_window config (#17478)
  hide S3 source connector from catalog (#17472)
  🪟 Migrate styles for Connection-related Components (#17339)
  Added new title (#17480)
  Refactor & convert `PageTitle` to SCSS (#17139)
  updated releaseStage for zendesk-talk (#17477)
  [low-code] Apply log level to stream loggers (#17284)
  🐛 Source Salesforce: filter out objects not supported by the Bulk API (#17453)
  Source Marketo: certify GA (#17445)
  Update greenhouse paginator (#17429)
  Add some services start validation to acceptance_tests.sh (#17425)
  📖 Removes $ from terminal commands to allow direct copying. (#17467)
  migrate source GA connectors to per-stream states (2) (#17410)
  Source Klaviyo: bump CDK dependency (#17422)
  Source Pinterest: change releaseStage to GA (#17045)
  Source Pinterest: Set start_date dynamically based on API restrictions for lookup (#17387)
  updated releaseStage to generally_available (#17374)
  ...
@octavia-squidington-iii octavia-squidington-iii mentioned this pull request Oct 5, 2022
Merged
jhammarstedt pushed a commit to jhammarstedt/airbyte that referenced this pull request Oct 31, 2022
@pedroslopez @jhammarstedt
Source Klaviyo: bump CDK dependency (airbytehq#17422)
c4a0086
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
area/connectors Connector related issues area/documentation Improvements or additions to documentation connectors/source/klaviyo
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@pedroslopez @octavia-squidington-iv @octavia-squidington-iii