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

Remove unused code path in HandleBotResponseAsync() #825

Merged
merged 2 commits into from
Mar 1, 2024
Merged

Remove unused code path in HandleBotResponseAsync() #825

merged 2 commits into from
Mar 1, 2024

Conversation

glahaye
Copy link
Collaborator

@glahaye glahaye commented Mar 1, 2024

Motivation and Context

Some dead wood is present in our code due to churn.

Description

Remove dead paths.

Contribution Checklist

@github-actions github-actions bot added the webapi Pull requests that update .net code label Mar 1, 2024
@glahaye glahaye merged commit e3853b2 into microsoft:main Mar 1, 2024
7 checks passed
teamleader-dev pushed a commit to vlink-group/chat-copilot that referenced this pull request Oct 7, 2024
### Motivation and Context
Some dead wood is present in our code due to churn.

### Description
Remove dead paths.


### Contribution Checklist
- [ ] The code builds clean without any errors or warnings
- [ ] The PR follows the [Contribution
Guidelines](https://github.com/microsoft/chat-copilot/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/chat-copilot/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
webapi Pull requests that update .net code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant