Skip to content

Comments

Python: Quick Redis sample fix#4066

Merged
giles17 merged 2 commits intomicrosoft:mainfrom
giles17:redis_store_session
Feb 19, 2026
Merged

Python: Quick Redis sample fix#4066
giles17 merged 2 commits intomicrosoft:mainfrom
giles17:redis_store_session

Conversation

@giles17
Copy link
Contributor

@giles17 giles17 commented Feb 19, 2026

Motivation and Context

Description

Contribution Checklist

  • The code builds clean without any errors or warnings
  • The PR follows the Contribution Guidelines
  • All unit tests pass, and I have added new tests where possible
  • Is this a breaking change? If yes, add "[BREAKING]" prefix to the title of the PR.

Copilot AI review requested due to automatic review settings February 19, 2026 04:51
@github-actions github-actions bot changed the title Quick Redis sample fix Python: Quick Redis sample fix Feb 19, 2026
@giles17 giles17 enabled auto-merge February 19, 2026 04:53
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This pull request refactors the Redis chat message store session sample to make the Redis URL configurable via environment variable instead of being hardcoded. This improves the sample's flexibility for users who need to connect to remote or authenticated Redis instances.

Changes:

  • Introduced a REDIS_URL constant that reads from the environment variable with a fallback to the local default
  • Replaced all hardcoded "redis://localhost:6379" occurrences with the REDIS_URL constant
  • Updated the prerequisites message to reflect the new configuration option

@giles17 giles17 added this pull request to the merge queue Feb 19, 2026
@giles17 giles17 removed this pull request from the merge queue due to a manual request Feb 19, 2026
@giles17 giles17 added this pull request to the merge queue Feb 19, 2026
@giles17 giles17 removed this pull request from the merge queue due to a manual request Feb 19, 2026
@giles17 giles17 enabled auto-merge February 19, 2026 04:58
@giles17 giles17 added this pull request to the merge queue Feb 19, 2026
Merged via the queue into microsoft:main with commit 56e5a15 Feb 19, 2026
24 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants