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

Release/v0.2.0 #47

Merged
merged 22 commits into from
Jan 5, 2022
Merged
Changes from 1 commit
Commits
Show all changes
22 commits
Select commit Hold shift + click to select a range
d5c6026
Merge pull request #42 from palmetto/main
emekdahl-palmetto Dec 3, 2021
06abf5a
dbt deps in docker image by default
jakeberesford-palmetto Dec 7, 2021
be46e85
One pair of parens is enough.
jakeberesford-palmetto Dec 7, 2021
7b58627
Remove the --fast flag
jakeberesford-palmetto Dec 7, 2021
2dfd71e
Add new command to recompile dbt deps
jakeberesford-palmetto Dec 7, 2021
a258f5e
Run linting
jakeberesford-palmetto Dec 7, 2021
772f08d
We still want to run clean on palm cleanup
jakeberesford-palmetto Dec 7, 2021
c5fcfc9
Drop branch schemas before clean to ensure all branches are removed
jakeberesford-palmetto Dec 7, 2021
f38350b
add_persist should return an empty string unless not persist
jakeberesford-palmetto Dec 8, 2021
2f0aca4
Remove add_persist function
jakeberesford-palmetto Dec 8, 2021
6b17d2b
Hard pin the version of black we are using to lint, ensure version ma…
jakeberesford-palmetto Dec 8, 2021
9dc20e6
Add volume mount for deps directory to
jakeberesford-palmetto Dec 8, 2021
a6d9e27
remove cmd_deps - this no longer makes sense, to update deps, the dev…
jakeberesford-palmetto Dec 8, 2021
f463b99
Move this section up so the document hierachy makes sense again
jakeberesford-palmetto Dec 8, 2021
3dafe0c
Add documentation around the decisions we made for dbt deps
jakeberesford-palmetto Dec 8, 2021
efd2e98
Handle non-existing dbt_project.yml
jakeberesford-palmetto Dec 8, 2021
2ad1a69
Merge pull request #45 from jakeberesford-palmetto/feature/pin-black-…
jakeberesford-palmetto Dec 8, 2021
99fc6f8
Fix the linter
jakeberesford-palmetto Dec 8, 2021
62f10a8
Initial support for v1.0 in dbt_containerizer
jakeberesford-palmetto Dec 9, 2021
67779ee
Lint for fun and profit (and passing builds)
jakeberesford-palmetto Dec 9, 2021
ca6a2c6
Merge pull request #44 from jakeberesford-palmetto/feature/dbt-deps-i…
jakeberesford-palmetto Jan 5, 2022
7b78dc2
Bump version and add changelog for v0.2 release
jakeberesford-palmetto Jan 5, 2022
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Drop branch schemas before clean to ensure all branches are removed
Run dbt deps after dbt clean as we will always need to re-instlal deps at this point
jakeberesford-palmetto committed Dec 7, 2021

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
commit c5fcfc9938ae27192eeb31ffb5ea36e63c02413f
2 changes: 1 addition & 1 deletion palm/plugins/dbt/commands/cmd_cleanup.py
Original file line number Diff line number Diff line change
@@ -9,7 +9,7 @@
def cli(ctx):
"""Removes any artifacts from Snowflake related to the current branch."""

cmd = "dbt clean && dbt run-operation drop_branch_schemas"
cmd = "dbt run-operation drop_branch_schemas && dbt clean && dbt deps"
env_vars = dbt_env_vars(ctx.obj.palm.branch)
success, msg = ctx.obj.run_in_docker(cmd, env_vars)
click.secho(msg, fg="green" if success else "red")