Skip to content

refactor(securityinsights): migrate to unified folder structure#37644

Merged
qiaozha merged 13 commits intoAzure:mainfrom
zedy-wj:securityinsights-folder-structure-refactor
Jan 8, 2026
Merged

refactor(securityinsights): migrate to unified folder structure#37644
qiaozha merged 13 commits intoAzure:mainfrom
zedy-wj:securityinsights-folder-structure-refactor

Conversation

@zedy-wj
Copy link
Member

@zedy-wj zedy-wj commented Sep 23, 2025

This PR migrates the securityinsights specification to the new unified folder structure according to Azure folder structure guidelines.

Changes:

  • Consolidated TypeSpec and Swagger files under resource-manager/Microsoft.SecurityInsights/SecurityInsights
  • Consolidated TypeSpec and Swagger files under data-plane/ThreatIntelligence
  • Add data-plane/ThreatIntelligence/suppressions.yaml file.
  • Add resource-manager/Microsoft.SecurityInsights/SecurityInsights/suppressions.yaml file.
  • Fixed input-file paths in readme.md
  • Update path in cspell.yaml file.
  • Verified TypeSpec compilation works correctly

Benefits:

  • Unified structure for better maintainability
  • Aligns with Azure folder structure standards
  • Maintains backward compatibility
  • Supports future TypeSpec migration efforts

@github-actions
Copy link

github-actions bot commented Sep 23, 2025

Next Steps to Merge

Next steps that must be taken to merge this PR:


Comment generated by summarize-checks workflow run.

@github-actions
Copy link

github-actions bot commented Sep 23, 2025

API Change Check

APIView identified API level changes in this PR and created the following API reviews

Language API Review for Package
Swagger Microsoft.SecurityInsights-SecurityInsights

@github-actions github-actions bot added the BreakingChangeReviewRequired <valid label in PR review process>add this label when breaking change review is required label Sep 23, 2025
@zedy-wj
Copy link
Member Author

zedy-wj commented Sep 23, 2025

Added label to suppress the Swagger SemanticValidation ci error. This error is not thrown by the latest version.
image

@zedy-wj zedy-wj marked this pull request as ready for review September 24, 2025 01:55
@zedy-wj zedy-wj added ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review FolderMigrationV2 A specific label for PRs like folder structure migration into v2. labels Sep 24, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added no-recent-activity There has been no recent activity on this issue. and removed no-recent-activity There has been no recent activity on this issue. labels Oct 13, 2025
@qiaozha
Copy link
Member

qiaozha commented Oct 23, 2025

Hold on this PR for this time, there's some problem with the common folder need to be further verified.

@microsoft-github-policy-service microsoft-github-policy-service bot added no-recent-activity There has been no recent activity on this issue. and removed no-recent-activity There has been no recent activity on this issue. labels Nov 17, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added no-recent-activity There has been no recent activity on this issue. and removed no-recent-activity There has been no recent activity on this issue. labels Dec 8, 2025
@microsoft-github-policy-service microsoft-github-policy-service bot added no-recent-activity There has been no recent activity on this issue. and removed no-recent-activity There has been no recent activity on this issue. labels Dec 29, 2025
@qiaozha qiaozha added the PublishToCustomers Acknowledgement the changes will be published to Azure customers. label Jan 8, 2026
@qiaozha qiaozha merged commit 2e24333 into Azure:main Jan 8, 2026
43 of 50 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Approved-Avocado Approved-LintDiff Approved-ModelValidation Approved-SemanticValidation Approved-Suppression ARMReview ARMSignedOff <valid label in PR review process>add this label when ARM approve updates after review BreakingChange-Approved-Benign Changes are not breaking at the REST API level and have at most minor impact to generated SDKs. BreakingChange-Go-Sdk-Suppression BreakingChange-Go-Sdk-Suppression-Approved BreakingChange-Python-Sdk-Suppression BreakingChange-Python-Sdk-Suppression-Approved BreakingChangeReviewRequired <valid label in PR review process>add this label when breaking change review is required data-plane FolderMigrationV2 A specific label for PRs like folder structure migration into v2. new-api-version PublishToCustomers Acknowledgement the changes will be published to Azure customers. resource-manager SuppressionReviewRequired

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants

Comments