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

Pin boto3 dependencies in dev requirements as well #6037

Merged
merged 2 commits into from
May 17, 2023

Conversation

wlach
Copy link
Contributor

@wlach wlach commented May 13, 2023

This fixes the backend-unit-tests job in CI, where a too-recent version of boto3 was getting installed.

wlach added 2 commits May 13, 2023 17:20
Testing to see if this fixes CI, where it appears a more recent (too
recent) version of boto3 is being installed.
This takes almost no time, and helps debug dependency issues (a
longer-term fix might be to switch to pip-compile).
@wlach
Copy link
Contributor Author

wlach commented May 13, 2023

The frontend unit tests are still failing, that's addressed in #6038

@arikfr arikfr merged commit e8071dc into getredash:master May 17, 2023
@wlach wlach deleted the pin-boto3-in-dev branch May 17, 2023 14:16
AkaashK pushed a commit to tharzeez/redash that referenced this pull request Jul 18, 2023
* Pin boto3 dependencies in dev requirements as well

Testing to see if this fixes CI, where it appears a more recent (too
recent) version of boto3 is being installed.

* Add circle step to list installed packages

This takes almost no time, and helps debug dependency issues (a
longer-term fix might be to switch to pip-compile).
harveyrendell pushed a commit to pushpay/redash that referenced this pull request Jan 8, 2025
* Pin boto3 dependencies in dev requirements as well

Testing to see if this fixes CI, where it appears a more recent (too
recent) version of boto3 is being installed.

* Add circle step to list installed packages

This takes almost no time, and helps debug dependency issues (a
longer-term fix might be to switch to pip-compile).
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