Skip to content

Fix CI#6161

Merged
y0z merged 5 commits intooptuna:masterfrom
nabenabe0928:hotfix-ci
Jun 17, 2025
Merged

Fix CI#6161
y0z merged 5 commits intooptuna:masterfrom
nabenabe0928:hotfix-ci

Conversation

@nabenabe0928
Copy link
Copy Markdown
Contributor

Motivation

This PR fixes the CI.
The cause of the failure comes from the Python 3.8 drop in fakeredis.
See here.

Description of the changes

  • Add version constraint to fakeredis

@nabenabe0928 nabenabe0928 marked this pull request as ready for review June 17, 2025 03:17
@nabenabe0928
Copy link
Copy Markdown
Contributor Author

nabenabe0928 commented Jun 17, 2025

@HideakiImamura @y0z
Could you review this PR?

@nabenabe0928 nabenabe0928 added the CI Continuous integration. label Jun 17, 2025
@nabenabe0928 nabenabe0928 added this to the v4.5.0 milestone Jun 17, 2025
Copy link
Copy Markdown
Member

@HideakiImamura HideakiImamura left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

Copy link
Copy Markdown
Member

@y0z y0z left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@y0z y0z merged commit 021868b into optuna:master Jun 17, 2025
14 checks passed
@y0z y0z removed their assignment Jun 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CI Continuous integration.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants