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

.NET: Fix Broken Links #4910

Closed
wants to merge 18 commits into from
Closed

Conversation

…t-versions/get-secret-versions?view=rest-keyvault-secrets-7.4
…/KernelSyntaxExamples/Example60_AdvancedMethodFunctions.cs
…anticKernel.Abstractions/PromptTemplate/PromptTemplateConfig.cs
…ernel-api/src/main/java/com/microsoft/semantickernel/chatcompletion/ChatRequestSettings.java
…nectors/Connectors.Memory.Qdrant/Http/SecureHttpHandler.cs
@justindhillon justindhillon requested review from a team as code owners February 6, 2024 23:03
@shawncal shawncal added .NET Issue or Pull requests regarding .NET code python Pull requests for the Python Semantic Kernel java Issue or PR regarding Java code documentation labels Feb 6, 2024
@github-actions github-actions bot changed the title Fix Broken Links .Net: Fix Broken Links Feb 6, 2024
@github-actions github-actions bot changed the title .Net: Fix Broken Links Python: Fix Broken Links Feb 6, 2024
@github-actions github-actions bot changed the title Python: Fix Broken Links Java: Fix Broken Links Feb 6, 2024
@justindhillon
Copy link
Contributor Author

@microsoft-github-policy-service agree

@markwallace-microsoft markwallace-microsoft self-assigned this Feb 7, 2024
@dsgrieve dsgrieve changed the title Java: Fix Broken Links .NET: Fix Broken Links Feb 7, 2024
@markwallace-microsoft
Copy link
Member

@justindhillon apologies for the delay in responding but this PR has a merge conflict which needs to be resolved. If you are available to fix this please re-open the PR and make the changes.

github-merge-queue bot pushed a commit that referenced this pull request Mar 22, 2024
### Description

I used [link-inspector](https://github.com/justindhillon/link-inspector)
to find and fix broken links in this project. This is an updated PR of
#4910, which fixes the merge conflict. Here are the links I have fixed:

https://docs.microsoft.com/rest/api/keyvault/getsecrets/getsecrets -->
https://learn.microsoft.com/en-us/rest/api/keyvault/secrets/get-secret/get-secret

https://docs.microsoft.com/rest/api/keyvault/getkeys/getkeys -->
https://learn.microsoft.com/en-us/rest/api/keyvault/keys/get-keys/get-keys

https://docs.microsoft.com/rest/api/keyvault/encrypt/encrypt -->
https://learn.microsoft.com/en-us/rest/api/keyvault/keys/encrypt/encrypt

https://docs.microsoft.com/rest/api/keyvault/decrypt/decrypt -->
https://learn.microsoft.com/en-us/rest/api/keyvault/keys/decrypt/decrypt

https://docs.microsoft.com/rest/api/keyvault/createkey/createkey -->
https://learn.microsoft.com/en-us/rest/api/keyvault/keys/create-key/create-key


https://docs.microsoft.com/rest/api/keyvault/getsecretversions/getsecretversions
-->
https://learn.microsoft.com/en-us/rest/api/keyvault/secrets/get-secret-versions/get-secret-versions


https://github.com/microsoft/semantic-kernel/blob/main/python/semantic_kernel/planning/plan.py
-->
https://github.com/microsoft/semantic-kernel/blob/main/python/semantic_kernel/planners/plan.py


https://github.com/microsoft/semantic-kernel/blob/main/python/semantic_kernel/planning/sequential_planner/Plugins/SequentialPlanning/skprompt.txt
-->
https://github.com/microsoft/semantic-kernel/blob/main/samples/plugins/QAPlugin/Form/skprompt.txt



### Support my work

These links were found with
[link-inspector](https://github.com/justindhillon/link-inspector). If
you find this PR useful, give the repo a ⭐

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [X] The code builds clean without any errors or warnings
- [X] The PR follows the [SK Contribution
Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/semantic-kernel/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 😄

---------

Co-authored-by: Stuart Morris <89403118+StuartMorrisHitachi@users.noreply.github.com>
Co-authored-by: SergeyMenshykh <68852919+SergeyMenshykh@users.noreply.github.com>
Co-authored-by: Mark Wallace <127216156+markwallace-microsoft@users.noreply.github.com>
Co-authored-by: Roger Barreto <19890735+RogerBarreto@users.noreply.github.com>
Co-authored-by: Jadyn <jadyn.wong@live.com>
Co-authored-by: Evan Mattson <35585003+moonbox3@users.noreply.github.com>
Co-authored-by: Chris <66376200+crickman@users.noreply.github.com>
Co-authored-by: Eduard van Valkenburg <eavanvalkenburg@users.noreply.github.com>
LudoCorporateShark pushed a commit to LudoCorporateShark/semantic-kernel that referenced this pull request Aug 25, 2024
### Description

I used [link-inspector](https://github.com/justindhillon/link-inspector)
to find and fix broken links in this project. This is an updated PR of
microsoft#4910, which fixes the merge conflict. Here are the links I have fixed:

https://docs.microsoft.com/rest/api/keyvault/getsecrets/getsecrets -->
https://learn.microsoft.com/en-us/rest/api/keyvault/secrets/get-secret/get-secret

https://docs.microsoft.com/rest/api/keyvault/getkeys/getkeys -->
https://learn.microsoft.com/en-us/rest/api/keyvault/keys/get-keys/get-keys

https://docs.microsoft.com/rest/api/keyvault/encrypt/encrypt -->
https://learn.microsoft.com/en-us/rest/api/keyvault/keys/encrypt/encrypt

https://docs.microsoft.com/rest/api/keyvault/decrypt/decrypt -->
https://learn.microsoft.com/en-us/rest/api/keyvault/keys/decrypt/decrypt

https://docs.microsoft.com/rest/api/keyvault/createkey/createkey -->
https://learn.microsoft.com/en-us/rest/api/keyvault/keys/create-key/create-key


https://docs.microsoft.com/rest/api/keyvault/getsecretversions/getsecretversions
-->
https://learn.microsoft.com/en-us/rest/api/keyvault/secrets/get-secret-versions/get-secret-versions


https://github.com/microsoft/semantic-kernel/blob/main/python/semantic_kernel/planning/plan.py
-->
https://github.com/microsoft/semantic-kernel/blob/main/python/semantic_kernel/planners/plan.py


https://github.com/microsoft/semantic-kernel/blob/main/python/semantic_kernel/planning/sequential_planner/Plugins/SequentialPlanning/skprompt.txt
-->
https://github.com/microsoft/semantic-kernel/blob/main/samples/plugins/QAPlugin/Form/skprompt.txt



### Support my work

These links were found with
[link-inspector](https://github.com/justindhillon/link-inspector). If
you find this PR useful, give the repo a ⭐

### Contribution Checklist

<!-- Before submitting this PR, please make sure: -->

- [X] The code builds clean without any errors or warnings
- [X] The PR follows the [SK Contribution
Guidelines](https://github.com/microsoft/semantic-kernel/blob/main/CONTRIBUTING.md)
and the [pre-submission formatting
script](https://github.com/microsoft/semantic-kernel/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 😄

---------

Co-authored-by: Stuart Morris <89403118+StuartMorrisHitachi@users.noreply.github.com>
Co-authored-by: SergeyMenshykh <68852919+SergeyMenshykh@users.noreply.github.com>
Co-authored-by: Mark Wallace <127216156+markwallace-microsoft@users.noreply.github.com>
Co-authored-by: Roger Barreto <19890735+RogerBarreto@users.noreply.github.com>
Co-authored-by: Jadyn <jadyn.wong@live.com>
Co-authored-by: Evan Mattson <35585003+moonbox3@users.noreply.github.com>
Co-authored-by: Chris <66376200+crickman@users.noreply.github.com>
Co-authored-by: Eduard van Valkenburg <eavanvalkenburg@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation java Issue or PR regarding Java code .NET Issue or Pull requests regarding .NET code python Pull requests for the Python Semantic Kernel
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants