Skip to content

Conversation

ms-henglu
Copy link
Member

  • Rename microsoft.insights_components_analyticsItems to Microsoft.Insights_components_analyticsItems
  • Rename microsoft.alertsManagement_smartDetectorAlertRules to Microsoft.AlertsManagement_smartDetectorAlertRules

These directories had lowercase namespace prefixes which caused the sync-reference-doc tool to treat them as separate namespaces from their uppercase counterparts. This resulted in remarks.json files being overwritten, losing most samples during sync.

After this fix, all 19 Microsoft.Insights examples and all 3 Microsoft.AlertsManagement examples are now properly synced together into their respective remarks.json files.

Also updated output directory paths in GNUmakefile and sync-reference-doc/main.go from bicep-refdocs-generator to template-reference-generator.

- Rename microsoft.insights_components_analyticsItems to Microsoft.Insights_components_analyticsItems
- Rename microsoft.alertsManagement_smartDetectorAlertRules to Microsoft.AlertsManagement_smartDetectorAlertRules

These directories had lowercase namespace prefixes which caused the sync-reference-doc
tool to treat them as separate namespaces from their uppercase counterparts. This
resulted in remarks.json files being overwritten, losing most samples during sync.

After this fix, all 19 Microsoft.Insights examples and all 3 Microsoft.AlertsManagement
examples are now properly synced together into their respective remarks.json files.

Also updated output directory paths in GNUmakefile and sync-reference-doc/main.go
from bicep-refdocs-generator to template-reference-generator.
@ms-henglu ms-henglu requested a review from magodo October 15, 2025 04:26
@ms-henglu ms-henglu merged commit ec2098d into main Oct 20, 2025
10 checks passed
@ms-henglu ms-henglu deleted the fix-example-directory-casing branch October 20, 2025 02:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants