⚠ This page is served via a proxy. Original site: https://github.com
This service does not collect credentials or authentication data.
Skip to content

Conversation

@xinaizha
Copy link

Issue #, if available:

Description of changes:
Add trailing slash to namespace strings

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@xinaizha xinaizha requested a review from a team January 30, 2026 14:21
@xinaizha xinaizha deployed to manual-approval January 30, 2026 14:21 — with GitHub Actions Active
Copy link
Contributor

@jariy17 jariy17 left a comment

Choose a reason for hiding this comment

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

Could you expand on this PR with more details:

  • What is the problem?
  • How should we fix it?
  • How can we verify the fix works?
    Once we have this information, we can review your PR

@xinaizha
Copy link
Author

  1. The problem: We are using namespace with trailing slash as the standard/default convention, so we want to ensure all memory namespace examples consistently use trailing slashes for proper path parsing and prefix matching.
  2. To fix it: end each namespace with a trailing slash.
  3. Test: have done unit tests

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