Skip to content

Conversation

benmoriceau
Copy link
Contributor

@benmoriceau benmoriceau commented Aug 12, 2025

What

Make sure that we only check the bulk CDK version if we modify the bulk CDK only and not the old one.

How

Review guide

User Impact

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

@benmoriceau benmoriceau requested review from a team as code owners August 12, 2025 01:28
Copy link
Contributor

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

Helpful Resources

PR Slash Commands

Airbyte Maintainers (that's you!) can execute the following slash commands on your PR:

  • /format-fix - Fixes most formatting issues.
  • /bump-version - Bumps connector versions.
    • You can specify a custom changelog by passing changelog. Example: /bump-version changelog="My cool update"
    • Leaving the changelog arg blank will auto-populate the changelog from the PR title.
  • /run-cat-tests - Runs legacy CAT tests (Connector Acceptance Tests)
  • /build-connector-images - Builds and publishes a pre-release docker image for the modified connector(s).
  • /poe connector source-example lock - Run the Poe lock task on the source-example connector, committing the results back to the branch.
  • /poe source example lock - Alias for /poe connector source-example lock.
  • /poe source example use-cdk-branch my/branch - Pin the source-example CDK reference to the branch name specified.
  • /poe source example use-cdk-latest - Update the source-example CDK dependency to the latest available version.

📝 Edit this welcome message.

Copy link
Contributor

github-actions bot commented Aug 12, 2025

destination-hubspot Connector Test Results

44 tests   14 ✅  24s ⏱️
 5 suites  30 💤
 5 files     0 ❌

Results for commit 5e3ce9b.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Aug 12, 2025

destination-dev-null Connector Test Results

47 tests   15 ✅  40s ⏱️
 4 suites  32 💤
 4 files     0 ❌

Results for commit 5e3ce9b.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Aug 12, 2025

destination-customer-io Connector Test Results

48 tests   18 ✅  23s ⏱️
 7 suites  30 💤
 7 files     0 ❌

Results for commit 5e3ce9b.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Aug 12, 2025

destination-clickhouse Connector Test Results

 15 files   15 suites   2m 56s ⏱️
222 tests 181 ✅ 41 💤 0 ❌
242 runs  201 ✅ 41 💤 0 ❌

Results for commit 5e3ce9b.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Aug 12, 2025

source-mysql Connector Test Results

  8 files    8 suites   28s ⏱️
 51 tests  51 ✅ 0 💤 0 ❌
175 runs  175 ✅ 0 💤 0 ❌

Results for commit aa04744.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Aug 12, 2025

destination-mssql Connector Test Results

135 tests   117 ✅  7m 8s ⏱️
 12 suites   18 💤
 12 files      0 ❌

Results for commit aa04744.

♻️ This comment has been updated with latest results.

Copy link
Contributor

github-actions bot commented Aug 12, 2025

destination-s3 Connector Test Results

808 tests   459 ✅  2h 21m 42s ⏱️
 26 suites  349 💤
 26 files      0 ❌

Results for commit aa04744.

♻️ This comment has been updated with latest results.

Copy link
Contributor

destination-azure-blob-storage Connector Test Results

340 tests   220 ✅  52m 33s ⏱️
 13 suites  120 💤
 13 files      0 ❌

Results for commit 5e3ce9b.

@benmoriceau benmoriceau merged commit d682dc3 into master Aug 12, 2025
30 checks passed
@benmoriceau benmoriceau deleted the bmoric/fix-action-check-cdk branch August 12, 2025 17:42
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.

2 participants