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.

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

Update oracle adapter docs #5798

Merged
merged 15 commits into from
Jul 25, 2024
Merged

Update oracle adapter docs #5798

merged 15 commits into from
Jul 25, 2024

Conversation

aosingh
Copy link
Contributor

@aosingh aosingh commented Jul 17, 2024

What are you changing in this pull request and why?

This PR updates oracle adapter docs:

  • Simpler installation and configuration using the thin driver
  • Integration with conda environment manager for dbt Python models
  • Oracle Specific Configuration updates

Checklist

@aosingh aosingh requested review from dataders and a team as code owners July 17, 2024 22:14
Copy link

vercel bot commented Jul 17, 2024

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

Name Status Preview Updated (UTC)
docs-getdbt-com ✅ Ready (Inspect) Visit Preview Jul 25, 2024 8:34am

@github-actions github-actions bot added content Improvements or additions to content size: medium This change will take up to a week to address labels Jul 17, 2024
@github-actions github-actions bot added size: large This change will more than a week to address and might require more than one person and removed size: medium This change will take up to a week to address labels Jul 24, 2024

## Python Models using Oracle Autonomous Database (ADB-S)
## Python models using Oracle Autonomous Database (ADB-S)
Copy link
Contributor

Choose a reason for hiding this comment

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

hey @aosingh , thanks so much for opening this up and looks great! this section here is wrapped in a version block. since we only have v1.5 and higher on the docs site now, we can remove the versionblock now since it applies to all. wanted to get your thoughts on that?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Sure, we can remove the version block. It makes sense

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Let me remove the version block and update the PR

Copy link
Contributor

Choose a reason for hiding this comment

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

great stuff! removed the versionblocks for you @aosingh !

Copy link
Contributor

@mirnawong1 mirnawong1 left a comment

Choose a reason for hiding this comment

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

lgtm and made some small typo/tweaks! happy to merge this for you pending my question. thanks so much for opening this up @aosingh - looks great!

cc @dataders

@aosingh
Copy link
Contributor Author

aosingh commented Jul 24, 2024

Thanks @mirnawong1 for reviewing the docs and fixing the typos. This looks good.

@mirnawong1 mirnawong1 merged commit f128f4f into dbt-labs:current Jul 25, 2024
3 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
auto update content Improvements or additions to content size: large This change will more than a week to address and might require more than one person
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants