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

changed dbt_project.yml, changed reference to profile so core gets ma… #23

Merged
merged 2 commits into from
Nov 9, 2022

Conversation

tuvaforrest
Copy link
Contributor

…terialized as tables

Describe your changes

Testing

Checklist before requesting a review

  • I have performed a self-review of my code
  • My code follows style guidelines
  • I have commented my code as necessary
  • I have made corresponding changes to documentation
  • My code runs without errors
  • I have implemented primary key tests
  • Every model has a corresponding yml file
  • [] I have generated the DAG locally with dbt and updated the files in the docs/ path

Gif of how this PR makes you feel

@sarah-tuva
Copy link
Member

@tuvaforrest Oh, I see what's going on. Instead of changing the name under models section in the dbt_project.yml file can we update the project name at the top to "medicare_cclf_connector"? I think this should fix the issue.

@sarah-tuva sarah-tuva merged commit 101d9e8 into main Nov 9, 2022
@sarah-tuva sarah-tuva deleted the issue_fix branch November 15, 2022 18:31
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