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

Run check_if_can_write_profile before create_profile_using_project_profile_template [CT-67] [Backport 1.0.latest] #4447

Merged
merged 3 commits into from
Jan 27, 2022
Merged

Run check_if_can_write_profile before create_profile_using_project_profile_template [CT-67] [Backport 1.0.latest] #4447

merged 3 commits into from
Jan 27, 2022

Conversation

NiallRees
Copy link
Contributor

@NiallRees NiallRees commented Dec 7, 2021

Resolves #4375

Description

Checklist

  • I have signed the CLA
  • I have run this code in development and it appears to resolve the stated issue
  • This PR includes tests, or tests are not required/relevant for this PR
  • I have updated the CHANGELOG.md and added information about my change

@cla-bot cla-bot bot added the cla:yes label Dec 7, 2021
@@ -285,6 +285,8 @@ def run(self):
# just setup the user's profile.
fire_event(SettingUpProfile())
profile_name = self.get_profile_name_from_current_project()
if not self.check_if_can_write_profile(profile_name=profile_name):
Copy link
Contributor Author

Choose a reason for hiding this comment

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

🙈

@jtcohen6 jtcohen6 added this to the v1.0.2 milestone Jan 13, 2022
@leahwicz leahwicz added the jira label Jan 19, 2022
Copy link
Contributor

@iknox-fa iknox-fa left a comment

Choose a reason for hiding this comment

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

LGTM! Thanks for the contribution

Copy link
Contributor

@iknox-fa iknox-fa left a comment

Choose a reason for hiding this comment

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

Putting this on hold-- PR is fine, but I wasn't able to reproduce the issue locally with or without this PR. Additional conversation in linked ticket.

Edit: Resolved. Merging PR

@iknox-fa iknox-fa changed the title Run check_if_can_write_profile before create_profile_using_project_profile_template Run check_if_can_write_profile before create_profile_using_project_profile_template [CT-67] Jan 27, 2022
@iknox-fa iknox-fa changed the title Run check_if_can_write_profile before create_profile_using_project_profile_template [CT-67] Run check_if_can_write_profile before create_profile_using_project_profile_template [CT-67] [Backport 1.0.latest] Jan 27, 2022
@iknox-fa iknox-fa merged commit f373518 into dbt-labs:main Jan 27, 2022
@leahwicz leahwicz added backport 1.0.latest Tag for PR to be backported to the 1.0.latest branch and removed backport 1.0.latest Tag for PR to be backported to the 1.0.latest branch labels Jan 27, 2022
leahwicz pushed a commit that referenced this pull request Feb 1, 2022
…ofile_template [CT-67] [Backport 1.0.latest] (#4447)

* Run check_if_can_write_profile before create_profile_using_project_profile_template

* Changelog

Co-authored-by: Ian Knox <81931810+iknox-fa@users.noreply.github.com>
leahwicz added a commit that referenced this pull request Feb 1, 2022
…ofile_template [CT-67] [Backport 1.0.latest] (#4447) (#4658)

* Run check_if_can_write_profile before create_profile_using_project_profile_template

* Changelog

Co-authored-by: Ian Knox <81931810+iknox-fa@users.noreply.github.com>

Co-authored-by: Niall Woodward <niall@niallrees.com>
Co-authored-by: Ian Knox <81931810+iknox-fa@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport 1.0.latest Tag for PR to be backported to the 1.0.latest branch cla:yes jira
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] dbt init overwrite just appends to profile
4 participants