Skip to content

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

Merged
adchia merged 2 commits into
feast-dev:masterfrom
breno-costa:widen-redis-dependency
Sep 7, 2023
Merged

adchia merged 2 commits into
feast-dev:masterfrom
breno-costa:widen-redis-dependency

Conversation

@breno-costa

Copy link
Copy Markdown
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

shuchu commented Sep 6, 2023

Copy link
Copy Markdown
Collaborator

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

@adchia

adchia commented Sep 7, 2023

Copy link
Copy Markdown
Collaborator

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
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