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

Git providers supporting CI #6692

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

Git providers supporting CI #6692

wants to merge 29 commits into from

Conversation

nataliefiann
Copy link
Contributor

@nataliefiann nataliefiann commented Dec 19, 2024

What are you changing in this pull request and why?

I've created this PR with reference to this git issue: #3704 to help users clarify which Git providers support CI.

In the PR, I have updated the Continuous integration page, created a card on the Git configuaration page and linked the table I created in the CI doc to it. I also created and used a snippet from https://docs.getdbt.com/docs/deploy/ci-jobs#trigger-a-ci-job-with-the-api

Update - I've also added the snippet to the Import a project by git URL and Continuous integration jobs in dbt Cloud pages.

Closes: #3704

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 December 19, 2024 14:17
Copy link

vercel bot commented Dec 19, 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 11:34am

@github-actions github-actions bot added content Improvements or additions to content size: small This change will take 1 to 2 days to address labels Dec 19, 2024
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.

looks much better, thanks for linking the snippet in other pages @nataliefiann ! approved and one question that can be followed up with another pr - should the table include api functionality? eg how you can trigger a job via the cloud api: https://docs.getdbt.com/docs/deploy/ci-jobs#trigger-a-ci-job-with-the-api?

Copy link
Contributor

@joellabes joellabes left a comment

Choose a reason for hiding this comment

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

Content looks great, suggested a tweaked title but can be workshopped further as needed!


- If your git provider has a [native dbt Cloud integration](/docs/cloud/git/git-configuration-in-dbt-cloud), you can seamlessly set up [continuous integration (CI)](/docs/deploy/ci-jobs) jobs directly within dbt Cloud.

- For providers without native integration, you can still use the git clone method to import your git URL or leverage the [dbt Cloud Administrative API](/docs/dbt-cloud-apis/admin-cloud-api) to trigger a CI job to run.
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
- For providers without native integration, you can still use the git clone method to import your git URL or leverage the [dbt Cloud Administrative API](/docs/dbt-cloud-apis/admin-cloud-api) to trigger a CI job to run.
- For providers without native integration, you can still use the git clone method to import your git URL and leverage the [dbt Cloud Administrative API](/docs/dbt-cloud-apis/admin-cloud-api) to trigger a CI job to run.

@@ -0,0 +1,15 @@
## Git providers who support CI
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
## Git providers who support CI
## Availability of features by Git provider

Suggest a title like this instead, since we say that CI jobs are possible for any provider if you trigger it with the admin API.

Comment on lines +47 to +48
title="Git providers who support CI"
body="Learn which Git providers support CI"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
title="Git providers who support CI"
body="Learn which Git providers support CI"
title="Availability of CI features by Git provider"
body="Learn which Git providers have native support for Continuous Integration workflows"

Same comment as below - open to alternative phrasing on this though

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: small This change will take 1 to 2 days to address
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Clarify what git providers support CI
3 participants