Skip to content

docs: remove references to source templates in API tutorial - #77606

Merged
Ian Alton (ian-at-airbyte) merged 1 commit into
masterfrom
katmarkham-patch-2
Apr 30, 2026
Merged

docs: remove references to source templates in API tutorial#77606
Ian Alton (ian-at-airbyte) merged 1 commit into
masterfrom
katmarkham-patch-2

Conversation

@katmarkham

@katmarkham Kat Wilson (katmarkham) commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

What

How

Review guide

User Impact

Can this PR be safely reverted and rolled back?

  • YES 💚
  • NO ❌

@octavia-bot
octavia-bot Bot marked this pull request as draft April 30, 2026 00:14
@octavia-bot

octavia-bot Bot commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

Note

📝 PR Converted to Draft

More info...

Thank you for creating this PR. As a policy to protect our engineers' time, Airbyte requires all PRs to be created first in draft status. Your PR has been automatically converted to draft status in respect for this policy.

As soon as your PR is ready for formal review, you can proceed to convert the PR to "ready for review" status by clicking the "Ready for review" button at the bottom of the PR page.

To skip draft status in future PRs, please include [ready] in your PR title or add the skip-draft-status label when creating your PR.

@github-actions

github-actions Bot commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

👋 Greetings, Airbyte Team Member!

Here are some helpful tips and reminders for your convenience.

💡 Show Tips and Tricks

PR Slash Commands

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

  • 🛠️ Quick Fixes
    • /format-fix - Fixes most formatting issues.
    • /bump-version - Bumps connector versions, scraping changelog description from the PR title.
      • Bump types: patch (default), minor, major, major_rc, rc, promote.
      • The rc type is a smart default: applies minor_rc if stable, or bumps the RC number if already RC.
      • The promote type strips the RC suffix to finalize a release.
      • Example: /bump-version type=rc or /bump-version type=minor
    • /bump-progressive-rollout-version - Alias for /bump-version type=rc. Bumps with an RC suffix and enables progressive rollout.
  • ❇️ AI Testing and Review (internal link: AI-SDLC Docs):
    • /ai-prove-fix - Runs prerelease readiness checks, including testing against customer connections.
    • /ai-canary-prerelease - Rolls out prerelease to 5-10 connections for canary testing.
    • /ai-review - AI-powered PR review for connector safety and quality gates.
  • 🚀 Connector Releases:
    • /publish-connectors-prerelease - Publishes pre-release connector builds (tagged as {version}-preview.{git-sha}) for all modified connectors in the PR.
  • ☕️ JVM connectors:
    • /update-connector-cdk-version connector=<CONNECTOR_NAME> - Updates the specified connector to the latest CDK version.
      Example: /update-connector-cdk-version connector=destination-bigquery
  • 🐍 Python connectors:
    • /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.
  • ⚙️ Admin commands:
    • /force-merge reason="<REASON>" - Force merges the PR using admin privileges, bypassing CI checks. Requires a reason.
      Example: /force-merge reason="CI is flaky, tests pass locally"
📚 Show Repo Guidance

Helpful Resources

📝 Edit this welcome message.

@katmarkham

Kat Wilson (katmarkham) commented Apr 30, 2026

Copy link
Copy Markdown
Contributor Author

Ian Alton (@ian-at-airbyte) this needs a close review. Alexandre Girard (@girarda) said I should pull out references to source_templates, but unfortunately they are listed a lot in the expected responses so I wasn't entirely sure how to handle

@katmarkham
Kat Wilson (katmarkham) marked this pull request as ready for review April 30, 2026 00:15
@github-actions

github-actions Bot commented Apr 30, 2026

Copy link
Copy Markdown
Contributor

Deploy preview for airbyte-docs ready!

Project:airbyte-docs
Status: ✅  Deploy successful!
Preview URL:https://airbyte-docs-h1rbdadlr-airbyte-growth.vercel.app
Latest Commit:0583c77

Deployed with vercel-action

@ian-at-airbyte Ian Alton (ian-at-airbyte) changed the title Update add-connector.md docs: remove references to source templates in API tutorial Apr 30, 2026
ian-at-airbyte
Ian Alton (ian-at-airbyte) approved these changes Apr 30, 2026
View reviewed changes

@ian-at-airbyte Ian Alton (ian-at-airbyte) left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

These are the last remaining references to source templates in published docs that aren't autogenerated.

I'm approving this because I think the change is a good one, but there remain a number of references to source templates in the API itself that could make this confusing for people. Will discuss with Alex.

@ian-at-airbyte
Ian Alton (ian-at-airbyte) merged commit 3f9b522 into master Apr 30, 2026
56 of 58 checks passed
@ian-at-airbyte
Ian Alton (ian-at-airbyte) deleted the katmarkham-patch-2 branch April 30, 2026 02:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ian-at-airbyte Ian Alton (ian-at-airbyte) ian-at-airbyte approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@katmarkham @ian-at-airbyte