Skip to content

Conversation

dataders
Copy link

@dataders dataders commented Aug 8, 2025

What change does this PR introduce?

I've added

  1. re test arguments
  2. added flag to require that all test arguments are nested under config
  3. migrated the single test in the integration tests that's pertinent to the deprecation
  4. profile: 'default so that Fusion actually parses instead of erroring about a missing profile field
  5. dbt_internal_packages/ -> .gitignore
  6. rename path specifier keys that have been deprecated since dbt Core 1.0

alex-ilyichov and others added 5 commits April 7, 2022 22:36
relative links to absolute
Co-authored-by: Joe Markiewicz <74217849+fivetran-joemarkiewicz@users.noreply.github.com>
@b-per
Copy link

b-per commented Aug 8, 2025

Hi team. We can talk about it next week, but this should not be required as the latest release (that diverged from main some time ago) doesn't have parse warnings/issues today in Fusion.

@dataders dataders changed the base branch from main to releases/v0.4.latest August 8, 2025 11:13
config-version: 2
require-dbt-version: [">=1.3.0", "<2.0.0"]

require-dbt-version: [">=1.10.6"]
Copy link
Author

@dataders dataders Aug 8, 2025

Choose a reason for hiding this comment

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

@b-per maybe we still need a new version to remove the upper bound so that fusion works?

Suggested change
require-dbt-version: [">=1.10.6"]
require-dbt-version: [">=1.3.0"]

Copy link

Choose a reason for hiding this comment

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

Fusion doesn't look at those today (but might in the future)

Comment on lines +15 to +18
vars:
fivetran_utils:
dbt_utils_dispatch_list:
- fivetran_utils
Copy link
Author

Choose a reason for hiding this comment

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

Suggested change
vars:
fivetran_utils:
dbt_utils_dispatch_list:
- fivetran_utils

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.

5 participants