-
Notifications
You must be signed in to change notification settings - Fork 5k
Planetarycomputer dotnet sdk public #50909
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?
Conversation
…src/Properties/AssemblyInfo.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…tests/Scenario/GeoCatalogCollectionTests.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…o planetarycomputer_dotnet_sdk
…re/azure-sdk-for-net into planetarycomputer_dotnet_sdk
…src/Properties/AssemblyInfo.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
…tests/Scenario/GeoCatalogCollectionTests.cs Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.
Pull Request Overview
This PR introduces support for the PlanetaryComputer resource in the Azure .NET SDK by adding new API definitions, tests, CI configuration, and documentation.
- Added trigger definitions for PlanetaryComputer in the CI pipelines
- Introduced new SDK source files, test projects, and documentation to support the GeoCatalog resource
- Updated solution and build files to integrate the new resource
Reviewed Changes
Copilot reviewed 20 out of 61 changed files in this pull request and generated 1 comment.
Show a summary per file
File | Description |
---|---|
sdk/resourcemanager/ci.mgmt.yml | Added PlanetaryComputer trigger lines for CI and PR builds |
sdk/planetarycomputer/ci.mgmt.yml | New CI configuration for PlanetaryComputer package |
sdk/planetarycomputer/Azure.ResourceManager.PlanetaryComputer/tsp-location.yaml | Added API spec commit info for the PlanetaryComputer resource |
sdk/planetarycomputer/Azure.ResourceManager.PlanetaryComputer/tests/* | New test cases for CRUD operations on GeoCatalog resources |
sdk/planetarycomputer/Azure.ResourceManager.PlanetaryComputer/src/* | New SDK project files and implementation for PlanetaryComputer |
sdk/planetarycomputer/Azure.ResourceManager.PlanetaryComputer/README.md | Documentation updates introducing the PlanetaryComputer SDK |
.vscode/cspell.json | Updated spell-check configuration to include PlanetaryComputer files |
sdk/planetarycomputer/Azure.ResourceManager.PlanetaryComputer/README.md
Outdated
Show resolved
Hide resolved
API Change CheckAPIView identified API level changes in this PR and created the following API reviews |
…README.md Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
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.
Blocking merge until @ronniegeraghty has a chance to review and assign a central service label that can be tracked for reporting purposes.
@@ -159,6 +160,7 @@ | |||
"skus", | |||
"Somaxconn", | |||
"southcentralus", | |||
"spatio", |
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.
Please move this into an area specific to your service. This is not a general concept that we want other libraries using.
Contributing to the Azure SDK
Please see our CONTRIBUTING.md if you are not familiar with contributing to this repository or have questions.
For specific information about pull request etiquette and best practices, see this section.