Skip to content

Fix incorrect adapter names in Dockerfile READMEs#1733

Open
sunilp wants to merge 2 commits intodbt-labs:mainfrom
sunilp:fix/dockerfile-readme-adapter-names
Open

Fix incorrect adapter names in Dockerfile READMEs#1733
sunilp wants to merge 2 commits intodbt-labs:mainfrom
sunilp:fix/dockerfile-readme-adapter-names

Conversation

@sunilp
Copy link

@sunilp sunilp commented Mar 10, 2026

Summary

The Docker READMEs for dbt-bigquery, dbt-redshift, and dbt-spark incorrectly reference "Snowflake" in example descriptions, even though the build targets and commands correctly reference their respective adapters.

This PR replaces "Snowflake" with the correct adapter name in each file:

  • dbt-bigquery/docker/README.md → "BigQuery"
  • dbt-redshift/docker/README.md → "Redshift"
  • dbt-spark/docker/README.md → "Spark"

Fixes #1700

Checklist

  • I have signed the CLA
  • This PR fixes a documentation bug (no code changes)

The Docker READMEs for dbt-bigquery, dbt-redshift, and dbt-spark
incorrectly reference "Snowflake" in their example descriptions.
Replace with the correct adapter names.

Fixes dbt-labs#1700
@sunilp sunilp requested a review from a team as a code owner March 10, 2026 01:57
@cla-bot
Copy link

cla-bot bot commented Mar 10, 2026

Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.

In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR.

CLA has not been signed by users: @sunilp

@cla-bot cla-bot bot added the cla:yes The PR author has signed the CLA label Mar 14, 2026
@sunilp
Copy link
Author

sunilp commented Mar 14, 2026

Thanks for your pull request, and welcome to our community! We require contributors to sign our Contributor License Agreement and we don't seem to have your signature on file. Check out this article for more information on why we have a CLA.

In order for us to review and merge your code, please submit the Individual Contributor License Agreement form attached above above. If you have questions about the CLA, or if you believe you've received this message in error, please reach out through a comment on this PR.

CLA has not been signed by users: @sunilp

already have signed

@sunilp
Copy link
Author

sunilp commented Mar 16, 2026

Friendly bump — CLA is signed and verified (status check passing). The CI check-run-approval step looks like it needs a maintainer to approve the workflow run for first-time contributors. Would appreciate a review when you get a chance!

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

Labels

cla:yes The PR author has signed the CLA

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Bug] Dockerfile READMEs refer to Snowflake instead of Redshift, BigQuery, and Spark

1 participant