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

Bump redis to latest version #539

Merged
merged 1 commit into from
Mar 29, 2023
Merged

Bump redis to latest version #539

merged 1 commit into from
Mar 29, 2023

Conversation

bmerry
Copy link
Contributor

@bmerry bmerry commented Mar 29, 2023

We're not actually using redis (it gets sucked in via katsdptelstate), but this will reduce the warnings from dependabot.

Note that the fix added in 4.5.3 is
incomplete, so this won't completely silence dependabot.

Checklist (if not applicable, edit to add (N/A) and mark as done):

  • If dependencies are added/removed: update setup.cfg and .pre-commit-config.yaml
  • (N/A) If modules are added/removed: use sphinx-apidoc to update files in doc/
  • (N/A) Ensure copyright notices are present and up-to-date
  • (N/A) If qualification tests are changed: attach a sample qualification report
  • (N/A) If design has changed: ensure documentation is up to date
  • (N/A) If ICD-defined sensors have been added: update fake_servers.py in katsdpcontroller to match

We're not actually using redis (it gets sucked in via katsdptelstate),
but this will reduce the warnings from dependabot.

Note that the fix added in 4.5.3 is
[incomplete](redis/redis-py#2665), so this
won't completely silence dependabot.
@bmerry bmerry requested a review from james-smith-za March 29, 2023 08:12
@james-smith-za
Copy link
Contributor

I kicked Jenkins to redo the build. Let's just wait for it to approve before we merge.

@bmerry bmerry merged commit 4541a3b into main Mar 29, 2023
@bmerry bmerry deleted the bump-redis branch March 29, 2023 08:51
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