Skip to content

Conversation

@dnskr
Copy link
Contributor

@dnskr dnskr commented Nov 8, 2025

Description

The PR fixes default values in the documentation for models_relative_path, seeds_relative_path and snapshots_relative_path.

Current values are taken from:

:param models_relative_path: The relative path to the dbt models directory within the project. Defaults to models
:param seeds_relative_path: The relative path to the dbt seeds directory within the project. Defaults to seeds
:param snapshots_relative_path: The relative path to the dbt snapshots directory within the project. Defaults to

Before changes:
image

After changes:
image

Breaking Change?

No

Checklist

  • I have made corresponding changes to the documentation (if required)
  • I have added tests that prove my fix is effective or that my feature works

Copy link
Collaborator

@tatiana tatiana left a comment

Choose a reason for hiding this comment

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

Well spot, @dnskr , it makes sense to change those values based on the dbt documentation:

Since we implemented this feature, dbt now supports a list of paths instead of a single path.

Since this would be a breaking change, I've logged a ticket (#2100) for Cosmos 2.0 so we support a list of paths instead of single path.

@codecov
Copy link

codecov bot commented Nov 12, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 97.81%. Comparing base (975ebae) to head (5f216a4).
⚠️ Report is 10 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2092   +/-   ##
=======================================
  Coverage   97.81%   97.81%           
=======================================
  Files          92       92           
  Lines        5951     5951           
=======================================
  Hits         5821     5821           
  Misses        130      130           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@netlify
Copy link

netlify bot commented Nov 18, 2025

Deploy Preview for sunny-pastelito-5ecb04 ready!

Name Link
🔨 Latest commit 996fcc9
🔍 Latest deploy log https://app.netlify.com/projects/sunny-pastelito-5ecb04/deploys/691c5446ceeee800086c811e
😎 Deploy Preview https://deploy-preview-2092--sunny-pastelito-5ecb04.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify project configuration.

@dnskr dnskr marked this pull request as draft November 18, 2025 21:33
@dnskr dnskr force-pushed the fix-doc-default-values branch from 0d087f1 to 2f3fa07 Compare November 22, 2025 22:33
@dnskr dnskr changed the title Fix default values in documentation docs: Fix default values in documentation Nov 22, 2025
@dnskr dnskr marked this pull request as ready for review November 22, 2025 22:36
@dnskr dnskr requested a review from tatiana November 22, 2025 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants