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

chore: Widen redis requirement to make it easier to install alongside other requirements #3755

Merged
merged 2 commits into from
Sep 7, 2023
Merged

chore: Widen redis requirement to make it easier to install alongside other requirements #3755

merged 2 commits into from
Sep 7, 2023

Conversation

breno-costa
Copy link
Contributor

What this PR does / why we need it: This PR widen Redis requirement to make it easier to install alongside other requirements, as well as to make it possible to fix some vulnerabilities.

Which issue(s) this PR fixes: #3753

Fixes #3753

@breno-costa breno-costa changed the title chore: widen redis requirement to make it easier to install alongside other requirements chore: Widen redis requirement to make it easier to install alongside other requirements Sep 1, 2023
@shuchu
Copy link
Collaborator

shuchu commented Sep 6, 2023

We may need to update the files under "sdk/python/requirements/" as well.

@adchia
Copy link
Collaborator

adchia commented Sep 7, 2023

Saw you left a message in the Slack about this @shuchu thanks!

For future reference, these files are updated via the make lock-python-ci-dependencies PYTHON=3.9 etc commands

Updated them in this PR

@adchia adchia enabled auto-merge (squash) September 7, 2023 05:04
@adchia adchia merged commit 0049356 into feast-dev:master Sep 7, 2023
22 checks passed
james-crabtree-sp pushed a commit to sailpoint/feast that referenced this pull request Sep 14, 2023
… other requirements (feast-dev#3755)

* chore: wide redis dependency to make it easier to install alongside other requirements

Signed-off-by: Breno Costa <[email protected]>

* update requirements

Signed-off-by: Danny C <[email protected]>

---------

Signed-off-by: Breno Costa <[email protected]>
Signed-off-by: Danny C <[email protected]>
Co-authored-by: Danny C <[email protected]>
james-crabtree-sp pushed a commit to sailpoint/feast that referenced this pull request Sep 14, 2023
… other requirements (feast-dev#3755)

* chore: wide redis dependency to make it easier to install alongside other requirements

Signed-off-by: Breno Costa <[email protected]>

* update requirements

Signed-off-by: Danny C <[email protected]>

---------

Signed-off-by: Breno Costa <[email protected]>
Signed-off-by: Danny C <[email protected]>
Co-authored-by: Danny C <[email protected]>
zseta pushed a commit to zseta/feast that referenced this pull request Feb 7, 2024
… other requirements (feast-dev#3755)

* chore: wide redis dependency to make it easier to install alongside other requirements

Signed-off-by: Breno Costa <[email protected]>

* update requirements

Signed-off-by: Danny C <[email protected]>

---------

Signed-off-by: Breno Costa <[email protected]>
Signed-off-by: Danny C <[email protected]>
Co-authored-by: Danny C <[email protected]>
Signed-off-by: Attila Toth <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Redis version in setup.py contains fixable vulnerabilities
3 participants