Move azure-ai-projects source code to a new service folder \sdk\aiprojects #44467
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
dargilco
requested review from
a team,
benbp,
glharper,
howieleung,
kashifkhan,
kingernupur,
lirenhe,
lmazuel,
msyyc,
nick863,
scbedd,
trangevi,
trrwilson,
weshaggard and
xiangyan99
as code owners
Contributor
There was a problem hiding this comment.
Pull request overview
This PR restructures the azure-ai-projects package by moving it from sdk/ai/azure-ai-projects/ to a new dedicated service directory sdk/aiprojects/. This reorganization includes updating CI/CD pipeline configurations, build artifacts, documentation settings, and code ownership mappings to reflect the new location.
Key changes:
- Created new CI/CD pipeline files (
ci.yml,tests.yml,platform-matrix.json) in thesdk/aiprojectsdirectory - Updated existing configuration files to reference the new
sdk/aiprojectspath instead ofsdk/ai/azure-ai-projects - Reorganized CODEOWNERS to group AI Projects ownership separately from the general AI SDK section
Reviewed changes
Copilot reviewed 8 out of 280 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| sdk/aiprojects/tests.yml | New weekly analysis pipeline configuration for the aiprojects service directory |
| sdk/aiprojects/platform-matrix.json | New platform matrix configuration with PortAudio installation requirement |
| sdk/aiprojects/ci.yml | New CI pipeline configuration for building and testing azure-ai-projects in the new location |
| sdk/ai/ci.yml | Removed azure-ai-projects artifact from the ai service directory CI pipeline |
| sdk/ai/azure-ai-projects/tsp-location.yaml | Deleted TypeSpec location file from old directory |
| eng/.docsettings.yml | Updated test path exclusion to reference new aiprojects directory |
| conda/conda-recipes/azure-ai-projects/meta.yaml | Updated home URL to point to new repository location |
| .github/CODEOWNERS | Moved AI Projects ownership entry to sdk/aiprojects with expanded owner list |
Comments suppressed due to low confidence (2)
sdk/aiprojects/ci.yml:1
- The trigger and PR paths reference
sdk/ai/andsdk/core/, but the service has been moved tosdk/aiprojects/. These paths should be updated to includesdk/aiprojects/to ensure the CI pipeline triggers correctly for changes in the new directory.
sdk/aiprojects/ci.yml:1 - The PR trigger paths reference
sdk/ai/andsdk/core/, but should includesdk/aiprojects/to match the new service directory location.
…jects-folder-move
…ypy311 and if it DOES, then cool no custom matrix necessary. OTHERWISE I'll add another commit that targets a platform matrix that doesn't include it.
…ub.com/Azure/azure-sdk-for-python into dargilco/azure-ai-projects-folder-move
Member
|
/azp run automation - pipeline-generation |
|
No pipelines are associated with this pull request. |
Member
|
/azp run automation - pipeline-generation-single |
|
No pipelines are associated with this pull request. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.