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

New sk-python-azure-functions-chatgpt-plugin and other minor updates #87

Merged
merged 4 commits into from
Dec 1, 2023

Conversation

raffertyuy
Copy link
Contributor

Motivation, Context and Description

This pull request contains 3 contributions, sorted by complexity

  1. Updated README.md of sk-python-flask-chatgpt-plugin
  • Simply added instructions for running using command line.
  • Previous instructions were only VSCode debugging.
  1. Added HTTP GET /logo.png to sk-csharp-chatgpt-plugin
  • Updated the .csproj and added Logo.cs and logo.png (using SK's logo).
  • ai-plugin.json refers to this, and was previously a broken image when testing with chat-copilot
  1. Adding a new sk-python-azure-functions-chatgpt-plugin
  • This was my feature request in Feature Request: Python Azure Functions ChatGPT Plugin Starter #85 which I went ahead to implement.
  • The code runs similar to other Azure functions using func host start.
  • The complication in this code is that there is no automated generator for either openapi.yaml or swagger.json, so this had to be manually created.

Contribution Checklist

@raffertyuy
Copy link
Contributor Author

@mkarle wonder if you can review this pull request?

Copy link

@moonbox3 moonbox3 left a comment

Choose a reason for hiding this comment

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

LGTM

@markwallace-microsoft markwallace-microsoft merged commit 7ce99cc into microsoft:main Dec 1, 2023
4 checks passed
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.

3 participants