Skip to content

Conversation

@fivetran-sheringuyen
Copy link
Contributor

@fivetran-sheringuyen fivetran-sheringuyen commented Oct 13, 2022

This pull request was created automatically 🎉
Before merging this PR (refer to Detailed Update Sheet 10/2022 for more information):

  • Verify dbt_project.yml & integration_tests/dbt_project.yml versions are properly bumped
  • Spot check for dispatch updates, dbt_utils.macro -> dbt.macro
  • Verify that .circleci directory has been removed
  • Verify integration_tests/requirements.txt adapters have been updated to 1.3.0 and dbt-databricks is added
  • Verify .buildkite directory has been added with the following: hooks/pre-command, scripts/run_models.sh, pipeline.yml
  • Update packages.yml, will need to bump source package (FT utils should be bumped)
  • Update .buildkite/scripts/run_models.sh with vars as applicable, if N/A then remove relevant lines from script
  • Remove databricks block from .buildkite/pipeline.yml if package is incompatible
  • Update schema names in integration_tests/ci/sample.profiles.yml
  • Update "spark" strings where applicable
  • Update CHANGELOG template and remove surrogate keys if not applicable to package
  • Update README for dbt version badge, install package version range and dependencies for: Fivetran_utils, dbt-utils and source packages
  • Regenerate docs
  • Follow Instructions for adding Buildkite
  • Follow instructions for removing Circleci

@fivetran-catfritz fivetran-catfritz force-pushed the MagicBot/dbt-utils-cross-db-migration branch from 73ae276 to c254634 Compare October 28, 2022 15:04
@fivetran-joemarkiewicz fivetran-joemarkiewicz added the update_type:dependency Primary focus is to update dependencies label Dec 20, 2022
Copy link
Contributor

@fivetran-joemarkiewicz fivetran-joemarkiewicz left a comment

Choose a reason for hiding this comment

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

lgtm

@fivetran-joemarkiewicz fivetran-joemarkiewicz merged commit b8d1d00 into main Dec 20, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

update_type:dependency Primary focus is to update dependencies

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants