-
Notifications
You must be signed in to change notification settings - Fork 1.6k
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
fix pip upgrade step in CI for windows #5320
Conversation
Thank you for your pull request! We could not find a changelog entry for this change. For details on how to document a change, see the contributing guide. |
a557a64
to
0bd05aa
Compare
e3bdce9
to
ada0601
Compare
@@ -9,7 +9,7 @@ endif | |||
.PHONY: dev | |||
dev: ## Installs dbt-* packages in develop mode along with development dependencies. | |||
@\ | |||
pip install -r dev-requirements.txt -r editable-requirements.txt && \ | |||
pip install -r dev-requirements.txt -r editable-requirements.txt |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@kwigley any reason why this isn't python -m pip
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I added that because it fixes an error I introduced a week ago (dangling & ). It's not actually used by CI.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
resolves #5321
Description
Fixes CI failures running integration tests in Windows.
Checklist
changie new
to create a changelog entry