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

Adding open ai file and fixing grammer #529

Merged

Conversation

Inder24
Copy link
Contributor

@Inder24 Inder24 commented Jul 8, 2024

In this PR,
I corrected some grammar for 2 readme.md files
Also for module 11, I corrected the path of python notebook(with image link updated) as well as added the notebook for OpenAI use case

Copy link

github-actions bot commented Jul 8, 2024

👋 Thanks for contributing @Inder24! We will review the pull request and get back to you soon.

Copy link

github-actions bot commented Jul 8, 2024

Check Broken Paths

We have automatically detected the following broken relative paths in your files.
Review and fix the paths to resolve this issue.

Check the file paths and associated broken paths inside them. For more details, check our Contributing Guide.

File Full Path Issues
./11-integrating-with-function-calling/translations/ko/README.md 1. ../../aoai-assignment.ipynb

Copy link

github-actions bot commented Jul 8, 2024

Check Broken Paths

We have automatically detected the following broken relative paths in your files.
Review and fix the paths to resolve this issue.

Check the file paths and associated broken paths inside them. For more details, check our Contributing Guide.

File Full Path Issues
./11-integrating-with-function-calling/translations/ko/README.md 1. ../../aoai-assignment.ipynb

@Inder24
Copy link
Contributor Author

Inder24 commented Jul 14, 2024

@koreyspace - let me know if anything is missing in this PR. I found some more notebook missing in this module so I'll add that in the next PR.

Copy link
Collaborator

@koreyspace koreyspace left a comment

Choose a reason for hiding this comment

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

Thanks for the PR @Inder24 !

@koreyspace koreyspace merged commit f9c3638 into microsoft:main Jul 28, 2024
6 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants