Skip to content

Commit

Permalink
dev: removes shandy-sqlfmt and adds sqlfluff + dbt to deps and CI (et…
Browse files Browse the repository at this point in the history
…/somenergia-jardiner!91)

Merge branch 'feature/implements-sqlfluff-as-formatter' into 'main'
  • Loading branch information
diegoquintanav committed Feb 7, 2024
2 parents 74d1cda + 052b6c9 commit 848ad28
Show file tree
Hide file tree
Showing 4 changed files with 323 additions and 333 deletions.
2 changes: 1 addition & 1 deletion .gitlab-ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,7 +48,7 @@ include:

pre-commit:
variables:
SKIP: "sqlfluff-lint,sqlfluff-fix"
SKIP: "sqlfluff-lint"
before_script:
- git config --global --add safe.directory "$CI_PROJECT_DIR"
tags:
Expand Down
2 changes: 1 addition & 1 deletion .gitlab-ci/.pre-commit.yml
Original file line number Diff line number Diff line change
Expand Up @@ -75,7 +75,7 @@ pre-commit:
git status
fi
fi
pre-commit run --all-files # check the status passes now with autofix otherwise, it'll bail here
"$PRE_COMMIT_ENV_ARGS" pre-commit run --all-files # check the status passes now with autofix otherwise, it'll bail here
else
exit 0 # we're all good, pipeline can proceed
fi
Expand Down
Loading

0 comments on commit 848ad28

Please sign in to comment.