-
Couldn't load subscription status.
- Fork 152
Update change-analysis-migration.md #186
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
Changed resource type from "Microsoft.Resources/resources" to "Microsoft.ResourceGraph/resourceChanges", thinking we have a typo here.
|
Learn Build status updates of commit a96f9ce: ✅ Validation status: passed
For more details, please refer to the build report. |
|
Can you review the proposed changes? Important: When the changes are ready for publication, adding a #label:"aq-pr-triaged" |
|
@sehoff16 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
@sehoff16 : Thanks for your contribution! The author(s) and reviewer(s) have been notified to review your proposed change. |
|
This pull request has been inactive for at least 14 days. If you are finished with your changes, don't forget to sign off. See the contributor guide for instructions. |
|
@hhunter-ms Could you review this proposed update to your article and enter Thanks! |
|
This pull request has been inactive for at least 14 days. If you are finished with your changes, don't forget to sign off. See the contributor guide for instructions. |
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 corrects a resource type reference in the Change Analysis migration documentation from an incorrect Microsoft.Resources/resources to the correct Microsoft.ResourceGraph/resourceChanges.
- Fixes the resource type specification in a comparison table between Change Analysis classic and the new Resource Graph-powered API
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
| | Web app in-app data (app settings, file changes, environmental variables) | Yes | No | | ||
| | Automatic onboarding for data collection | No | Yes | | ||
| | Resource type | `Microsoft.ChangeAnalysis/changes` and `Microsoft.ChangeAnalysis/ resourcechanges` | `Microsoft.Resources/ resources` | | ||
| | Resource type | `Microsoft.ChangeAnalysis/changes` and `Microsoft.ChangeAnalysis/ resourcechanges` | `Microsoft.ResourceGraph/ resourceChanges` | |
Copilot
AI
Oct 2, 2025
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.
Inconsistent spacing in resource type names. There should be no space before the forward slash in Microsoft.ChangeAnalysis/ resourcechanges and Microsoft.ResourceGraph/ resourceChanges.
| | Resource type | `Microsoft.ChangeAnalysis/changes` and `Microsoft.ChangeAnalysis/ resourcechanges` | `Microsoft.ResourceGraph/ resourceChanges` | | |
| | Resource type | `Microsoft.ChangeAnalysis/changes` and `Microsoft.ChangeAnalysis/resourcechanges` | `Microsoft.ResourceGraph/resourceChanges` | |
|
Can you review this old PR and determine whether it needs to be closed or merged? @MicrosoftDocs/public-repo-pr-review-team |
|
This pull request has been inactive for at least 14 days. If you are finished with your changes, don't forget to sign off. See the contributor guide for instructions. |
Changed resource type from "Microsoft.Resources/resources" to "Microsoft.ResourceGraph/resourceChanges", thinking we have a typo here.