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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

Enable Qdrant connector nuget package #415

Merged
merged 2 commits into from
Apr 11, 2023

Conversation

awharrison-28
Copy link
Contributor

Motivation and Context

Ready to release Qdrant connector nuget package. Uncommented import of nuget package properties to enable package generation.

Contribution Checklist

@awharrison-28 awharrison-28 changed the title enable qdrant nuget - uncommented import of nuget package properties Enable Qdrant connector nuget package Apr 11, 2023
@adrianwyatt adrianwyatt enabled auto-merge (squash) April 11, 2023 21:13
@adrianwyatt adrianwyatt self-assigned this Apr 11, 2023
@adrianwyatt adrianwyatt merged commit da74d94 into microsoft:main Apr 11, 2023
dluc pushed a commit to dluc/semantic-kernel that referenced this pull request Apr 12, 2023
### Motivation and Context
Ready to release Qdrant connector nuget package. Uncommented import of
nuget package properties to enable package generation.
@awharrison-28 awharrison-28 deleted the enable_qdrant_nuget branch April 13, 2023 17:28
dehoward pushed a commit to lemillermicrosoft/semantic-kernel that referenced this pull request Jun 1, 2023
### Motivation and Context
Ready to release Qdrant connector nuget package. Uncommented import of
nuget package properties to enable package generation.
golden-aries pushed a commit to golden-aries/semantic-kernel that referenced this pull request Oct 24, 2023
### Motivation and Context

<!-- Thank you for your contribution to the chat-copilot repo!
Please help reviewers and future users, providing the following
information:
  1. Why is this change required?
  2. What problem does it solve?
  3. What scenario does it contribute to?
  4. If it fixes an open issue, please link to the issue here.
-->
this PR makes small changes which will be required in a follow-up PR
that removes the AAD env variables from the frontend.

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->
- [X] The code builds clean without any errors or warnings
- [X] 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
- [X] All unit tests pass, and I have added new tests where possible
- [X] I didn't break anyone 馃槃
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.

None yet

2 participants