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

Nfiann-prerelease #6502

Open
wants to merge 42 commits into
base: current
Choose a base branch
from
Open

Nfiann-prerelease #6502

wants to merge 42 commits into from

Conversation

nataliefiann
Copy link
Contributor

@nataliefiann nataliefiann commented Nov 18, 2024

What are you changing in this pull request and why?

I've created this PR following Git issue: #2144 This is one of two PRs - the second will be a new doc

This PR provides users with instructions on installing pre-releases.

Closes: #2144

Checklist

  • I have reviewed the Content style guide so my content adheres to these guidelines.
  • The topic I'm writing about is for specific dbt version(s) and I have versioned it according to the version a whole page and/or version a block of content guidelines.
  • I have added checklist item(s) to this list for anything anything that needs to happen before this PR is merged, such as "needs technical review" or "change base branch."
  • The content in this PR requires a dbt release note, so I added one to the release notes page.

🚀 Deployment available! Here are the direct links to the updated files:

@nataliefiann nataliefiann requested a review from a team as a code owner November 18, 2024 21:06
Copy link

vercel bot commented Nov 18, 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 Jan 9, 2025 9:47pm

@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 Nov 18, 2024
website/docs/docs/core/pip-install.md Outdated Show resolved Hide resolved
website/docs/docs/core/pip-install.md Outdated Show resolved Hide resolved
website/docs/docs/core/pip-install.md Outdated Show resolved Hide resolved
website/docs/docs/core/pip-install.md Outdated Show resolved Hide resolved
website/docs/docs/core/pip-install.md Outdated Show resolved Hide resolved
website/docs/docs/core/pip-install.md Outdated Show resolved Hide resolved
Copy link
Contributor

@dbeatty10 dbeatty10 left a comment

Choose a reason for hiding this comment

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

LGTM

website/docs/docs/core/pip-install.md Outdated Show resolved Hide resolved
@runleonarun
Copy link
Collaborator

runleonarun commented Jan 10, 2025

Hey @nataliefiann! I turned the following cleanup items into an issue #6753

Thanks for fixing those sneaky backticks! I noticed a few things that might need to be cleaned up in a follow up PR (or issue if you don't have time to work on a PR):

  • Can we delete this FAQ if this content is covered in this page?
    • Before making the previous change, look at where this FAQ is referenced to make sure it works on those pages.
  • The TOC to the right is wonky because of the FAQs. I suggest we downgrade the headings from H2 ## to H3 ### so they don't show up.
    • Before making the previous change, look at where those two FAQs are referenced to make sure it works on those pages.
  • Starting with "Using Virtual environments," upgrade the H3 to H2
  • Upgrade all other headings one level and upgrade "Activate your virtual environment" to an H2.

Copy link
Collaborator

@runleonarun runleonarun left a comment

Choose a reason for hiding this comment

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

Looks good to ship!

@runleonarun runleonarun mentioned this pull request Jan 10, 2025
6 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
content Improvements or additions to content size: medium This change will take up to a week to address
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Instructions how to install pre-releases
5 participants