Skip to content

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.

breno-costa and others added 2 commits September 6, 2023 22:02
…ther requirements

Signed-off-by: Breno Costa <brenocosta0901@gmail.com>
Signed-off-by: Danny C <d.chiao@gmail.com>
@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
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 <brenocosta0901@gmail.com>

* update requirements

Signed-off-by: Danny C <d.chiao@gmail.com>

---------

Signed-off-by: Breno Costa <brenocosta0901@gmail.com>
Signed-off-by: Danny C <d.chiao@gmail.com>
Co-authored-by: Danny C <d.chiao@gmail.com>
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 <brenocosta0901@gmail.com>

* update requirements

Signed-off-by: Danny C <d.chiao@gmail.com>

---------

Signed-off-by: Breno Costa <brenocosta0901@gmail.com>
Signed-off-by: Danny C <d.chiao@gmail.com>
Co-authored-by: Danny C <d.chiao@gmail.com>
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 <brenocosta0901@gmail.com>

* update requirements

Signed-off-by: Danny C <d.chiao@gmail.com>

---------

Signed-off-by: Breno Costa <brenocosta0901@gmail.com>
Signed-off-by: Danny C <d.chiao@gmail.com>
Co-authored-by: Danny C <d.chiao@gmail.com>
Signed-off-by: Attila Toth <hello@attilatoth.dev>
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