-
Notifications
You must be signed in to change notification settings - Fork 143
Oracledb Connector Updates for Python #331
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
base: main
Are you sure you want to change the base?
Oracledb Connector Updates for Python #331
Conversation
…efinition in the oracle-connector md file.
Update .NET info in oracle-connector.md
Merge upstream changes into oracledb-connector to keep branch upto date with main to avoid merge conflict.:wq
…ard similarity in C# connector. Also, updated the table formatting for Java connector.
Changed single quotes to double quotes to ensure correct C# syntax is used,
Change single quotes to double quotes
|
Learn Build status updates of commit 33ddabf: 💡 Validation status: suggestions
semantic-kernel/concepts/vector-store-connectors/out-of-the-box-connectors/index.md
semantic-kernel/concepts/vector-store-connectors/out-of-the-box-connectors/oracle-connector.md
For more details, please refer to the build report. Note: Your PR may contain errors or warnings or suggestions unrelated to the files you changed. This happens when external dependencies like GitHub alias, Microsoft alias, cross repo links are updated. Please use these instructions to resolve them. |
|
@microsoft-github-policy-service agree company="Oracle" |
@microsoft-github-policy-service agree company="Oracle" |
|
@microsoft-github-policy-service agree company="Oracle" |
|
@westey-m We have added the Oracle Python Semantic Kernel doc. Hoping you could review and approve. |
|
@eavanvalkenburg @monita1208 and @alexkeh Tagging all of you as well. |
|
Learn Build status updates of commit 3b7e598: ✅ Validation status: passed
For more details, please refer to the build report. |
eavanvalkenburg
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
overall looks good, small nit on the python install.
semantic-kernel/concepts/vector-store-connectors/out-of-the-box-connectors/oracle-connector.md
Outdated
Show resolved
Hide resolved
|
Learn Build status updates of commit aba80d0: ✅ Validation status: passed
For more details, please refer to the build report. |
semantic-kernel/concepts/vector-store-connectors/out-of-the-box-connectors/oracle-connector.md
Outdated
Show resolved
Hide resolved
|
Learn Build status updates of commit 1d1ef6d: ✅ Validation status: passed
For more details, please refer to the build report. |
|
@westey-m Please review and approve the doc changes for Oracle Python Semantic Kernel doc. |
Updated Oracledb Connector for Python