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

Rename "MemoriesStore" to "MemoryStore" #138

Merged
merged 1 commit into from
Aug 9, 2023
Merged

Rename "MemoriesStore" to "MemoryStore" #138

merged 1 commit into from
Aug 9, 2023

Conversation

glahaye
Copy link
Collaborator

@glahaye glahaye commented Aug 9, 2023

Motivation and Context

Just making this grammatically correct in English. (It was driving me nuts)

Description

Rename "MemoriesStore" to "MemoryStore"

Contribution Checklist

@github-actions github-actions bot added webapp Pull requests that update Typescript code webapi Pull requests that update .net code deployment Issues related to deploying Chat-Copilot PR: ready for review labels Aug 9, 2023
@glahaye glahaye added this pull request to the merge queue Aug 9, 2023
Merged via the queue into microsoft:main with commit 6b6f369 Aug 9, 2023
7 checks passed
@glahaye glahaye deleted the memory_store branch August 9, 2023 21:24
teamleader-dev pushed a commit to vlink-group/chat-copilot that referenced this pull request Oct 7, 2024
### Motivation and Context
Just making this grammatically correct in English. (It was driving me
nuts)

### Description
Rename "MemoriesStore" to "MemoryStore"

### Contribution Checklist
- [ ] The code builds clean without any errors or warnings
- [ ] The PR follows the [Contribution
Guidelines](https://github.com/microsoft/copilot-chat/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/copilot-chat/blob/main/CONTRIBUTING.md#development-scripts)
raises no violations
- [ ] All unit tests pass, and I have added new tests where possible
- [ ] I didn't break anyone 😄
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deployment Issues related to deploying Chat-Copilot webapi Pull requests that update .net code webapp Pull requests that update Typescript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants