Skip to content

Comments

AJ's community PR test (do not merge)#60807

Draft
Aaron ("AJ") Steers (aaronsteers) wants to merge 13 commits intoairbytehq:masterfrom
aaronsteers:aj/dummy-pr/community-test
Draft

AJ's community PR test (do not merge)#60807
Aaron ("AJ") Steers (aaronsteers) wants to merge 13 commits intoairbytehq:masterfrom
aaronsteers:aj/dummy-pr/community-test

Conversation

@aaronsteers
Copy link
Collaborator

Demo/test PR of new Community CI workflows, including BYO Creds.

@vercel
Copy link

vercel bot commented May 20, 2025

Aaron ("AJ") Steers (@aaronsteers) is attempting to deploy a commit to the Airbyte Growth Team on Vercel.

A member of the Team first needs to authorize it.

@vercel
Copy link

vercel bot commented May 20, 2025

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

Name Status Preview Comments Updated (UTC)
airbyte-docs ✅ Ready (Inspect) Visit Preview 💬 Add feedback May 20, 2025 1:19am

@github-actions
Copy link
Contributor

👋 Welcome to Airbyte!

Thank you for your contribution from aaronsteers/airbyte! We're excited to have you in the Airbyte community.

Helpful Resources

PR Slash Commands

As needed or by request, Airbyte Maintainers can execute the following slash commands on your PR:

  • /format-fix - Fixes most formatting issues.
  • /bump-version - Bumps connector versions.
  • /run-connector-tests - Runs connector tests.

If you have any questions, feel free to ask in the PR comments or join our Slack community.

@aaronsteers
Copy link
Collaborator Author

/run-connector-tests

@aaronsteers
Copy link
Collaborator Author

/run-connector-tests

@aaronsteers
Copy link
Collaborator Author

/run-connector-tests

@aaronsteers
Copy link
Collaborator Author

Aaron ("AJ") Steers (aaronsteers) commented May 28, 2025

/run-connector-tests

Connector CI Tests job started... Check job output.
Connector CI Tests job completed. See logs for details.

@aaronsteers
Copy link
Collaborator Author

Aaron ("AJ") Steers (aaronsteers) commented May 28, 2025

/run-connector-tests

Connector CI Tests Started

These tests will leverage Airbyte's integration test credentials.

Check job output.
Connector CI Tests job completed. See logs for details.

@aaronsteers
Copy link
Collaborator Author

/run-cat-tests

@github-actions
Copy link
Contributor

👋 Welcome to Airbyte!

Thank you for your contribution from aaronsteers/airbyte! We're excited to have you in the Airbyte community.

Helpful Resources

PR Slash Commands

As needed or by request, Airbyte Maintainers can execute the following slash commands on your PR:

  • /format-fix - Fixes most formatting issues.
  • /bump-version - Bumps connector versions.
  • /run-connector-tests - Runs connector tests.
  • /run-cat-tests - Runs CAT tests.

If you have any questions, feel free to ask in the PR comments or join our Slack community.

Tips for Working with CI

  1. "Vercel" CI failures. You can always ignore these. These are our docs builds but they can only be run by approval. A maintainer will approve the workflow upon request, but this failure should not be of concern.
  2. Pre-Release Checks. Please pay attention to these, as they contain standard checks on the metadata.yaml file, docs requirements, etc. If you need help resolving a pre-release check, please ask a maintainer.
    • Note: If you are creating a new connector, please be sure to replace the default logo.svg file with a suitable icon.
  3. Connector CI Tests. Some failures here may be expected if your tests require credentials. Please review these results to ensure (1) unit tests are passing, if applicable, and (2) integration tests pass to the degree possible and expected.
  4. (Optional.) BYO Connector Credentials for tests in your fork. You can optionally set up your fork with BYO credentials for your connector. This can significantly speed up your review, ensuring your changes are fully tested before the maintainers begin their review.

Copilot AI review requested due to automatic review settings August 5, 2025 22:21
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull Request Overview

This is a demo/test PR for new Community CI workflows that includes dummy commits to test the workflow functionality. The PR adds temporary comment lines to connector metadata files for testing purposes.

  • Adds dummy comments to connector metadata files for CI workflow testing
  • Includes "todo: revert-me" notes indicating these are temporary changes for testing

Reviewed Changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
airbyte-integrations/connectors/source-klaviyo/metadata.yaml Adds dummy comment for CI workflow testing
airbyte-integrations/connectors/destination-teradata/metadata.yaml Adds dummy comment for CI workflow testing

@aaronsteers Aaron ("AJ") Steers (aaronsteers) marked this pull request as draft August 5, 2025 22:21
@github-actions
Copy link
Contributor

👋 Welcome to Airbyte!

Thank you for your contribution from aaronsteers/airbyte! We're excited to have you in the Airbyte community.

Helpful Resources

PR Slash Commands

As needed or by request, Airbyte Maintainers can execute the following slash commands on your PR:

  • /format-fix - Fixes most formatting issues.
  • /bump-version - Bumps connector versions.
  • /run-connector-tests - Runs connector tests.
  • /run-cat-tests - Runs CAT tests.
  • /build-connector-images - Builds and publishes a pre-release docker image for the modified connector(s).

If you have any questions, feel free to ask in the PR comments or join our Slack community.

Tips for Working with CI

  1. Pre-Release Checks. Please pay attention to these, as they contain standard checks on the metadata.yaml file, docs requirements, etc. If you need help resolving a pre-release check, please ask a maintainer.
    • Note: If you are creating a new connector, please be sure to replace the default logo.svg file with a suitable icon.
  2. Connector CI Tests. Some failures here may be expected if your tests require credentials. Please review these results to ensure (1) unit tests are passing, if applicable, and (2) integration tests pass to the degree possible and expected.
  3. (Optional.) BYO Connector Credentials for tests in your fork. You can optionally set up your fork with BYO credentials for your connector. This can significantly speed up your review, ensuring your changes are fully tested before the maintainers begin their review.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants