Skip to content

Conversation

@pvrk
Copy link
Contributor

@pvrk pvrk commented Feb 10, 2026

No description provided.

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 075bfb3:

✅ Validation status: passed

File Status Preview URL Details
.claude/settings.local.json ✅Succeeded
sharepoint/sharepoint-ps/Microsoft.Online.SharePoint.PowerShell/Get-SPODataAccessGovernanceInsight.md ✅Succeeded View (sharepoint-ps)
sharepoint/sharepoint-ps/Microsoft.Online.SharePoint.PowerShell/Start-SPODataAccessGovernanceInsight.md ✅Succeeded View (sharepoint-ps)

For more details, please refer to the build report.

@pvrk
Copy link
Contributor Author

pvrk commented Feb 10, 2026

@samkabue : This PR also has a local settings JSON along with the regular documentation updates. Can I just delete that file from this PR? Or should I raise a new PR? Please suggest the way forward. Thanks.

@samkabue
Copy link
Contributor

whichever is easier for you. @pvrk


In reply to: 3877970611

@pvrk
Copy link
Contributor Author

pvrk commented Feb 11, 2026

@samkabue : Deleted file from Github interface. Please check and let me know if more changes are required.

@samkabue
Copy link
Contributor

@samkabue : Deleted file from Github interface. Please check and let me know if more changes are required.

it's still there.

image

Aliases:
Accepted values: SharingLinks_Anyone, SharingLinks_PeopleInYourOrg, SharingLinks_Guests, SensitivityLabelForFiles, EveryoneExceptExternalUsersAtSite, EveryoneExceptExternalUsersForItems, PermissionedUsers, PermissionsReport
Accepted values: SharingLinks_Anyone, SharingLinks_PeopleInYourOrg, SharingLinks_Guests, SensitivityLabelForFiles, EveryoneExceptExternalUsersAtSite, EveryoneExceptExternalUsersForItems, PermissionedUsers, PermissionsReport, EveryoneExceptExternalUsers, Everyone

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd list these values in sorted order, for easy perusal

Aliases:
Accepted values: SharingLinks_Anyone, SharingLinks_PeopleInYourOrg, SharingLinks_Guests, SensitivityLabelForFiles, EveryoneExceptExternalUsersAtSite, EveryoneExceptExternalUsersForItems, PermissionedUsers, PermissionsReport
Accepted values: SharingLinks_Anyone, SharingLinks_PeopleInYourOrg, SharingLinks_Guests, SensitivityLabelForFiles, EveryoneExceptExternalUsersAtSite, EveryoneExceptExternalUsersForItems, PermissionedUsers, PermissionsReport, EveryoneExceptExternalUsers, Everyone

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd list these values in sorted order, for easy perusal

Type: Microsoft.Online.SharePoint.TenantAdministration.WorkloadEnum
Parameter Sets: (All)
Parameter Sets: EEEUParameterSet, SharingLinkParameterSet, LabelParameterSet, SitePermissionsParameterSet, UserPermissionsParameterSet
Aliases:
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'd list these values in sorted order, for easy perusal

This file should not be tracked in version control as it contains
local Claude Code settings specific to individual development environments.

Co-Authored-By: Claude Sonnet 4.5 <[email protected]>
@github-actions
Copy link

@xiaojhua-work
You've been added as a reviewer to this pull request because you're listed as a Content Lead for the product area or feature covered by one or more articles in this PR.

Before this PR can be merged, a Content Lead needs to add the Sign off label to it. Only one of the listed reviewers needs to add the Sign off label. Before you sign off a PR, confirm the following:

  • Article content has been checked for technical accuracy.
  • There are no errors or warnings in build validation.
  • There are no severity 1 or 2 PoliCheck issues.
  • All articles have a minimum Acrolinx score of 80.
  • All required PR checks at the bottom of the PR are passing.

After you've confirmed the above, do the following to sign off:

  1. Select the gear next to Labels.
  2. Select Sign off from the label list.
  3. Click away from the label list.

image

After you've signed off, our operations (PubOps) team will review the PR for issues that may impact formatting and the customer experience. If any changes are needed, they will return
the PR to the submitter with required changes. After they've completed those changes, you need to sign off again.

When the PR passes PubOps review, they'll merge the PR. If the PR is merged into the main branch, changes will go live within the next couple hours.

If you think you were incorrectly added as a reviewer, have questions about this process, or need help signing off on this PR, post a message to https://aka.ms/tcphelp.

@learn-build-service-prod
Copy link
Contributor

PoliCheck Scan Report

The following report lists PoliCheck issues in PR files. Before you merge the PR, you must fix all severity-1 and severity-2 issues. The AI Review Details column lists suggestions for either removing or replacing the terms. If you find a false positive result, mention it in a PR comment and include this text: #policheck-false-positive. This feedback helps reduce false positives in future scans.

✅ No issues found

More information about PoliCheck

Information: PoliCheck | Severity Guidance | Term
For any questions: Try searching the learn.microsoft.com contributor guides or post your question in the Learn support channel.

@learn-build-service-prod
Copy link
Contributor

Learn Build status updates of commit 962b1fe:

✅ Validation status: passed

File Status Preview URL Details
.gitignore ✅Succeeded
sharepoint/sharepoint-ps/Microsoft.Online.SharePoint.PowerShell/Get-SPODataAccessGovernanceInsight.md ✅Succeeded View (sharepoint-ps)
sharepoint/sharepoint-ps/Microsoft.Online.SharePoint.PowerShell/Start-SPODataAccessGovernanceInsight.md ✅Succeeded View (sharepoint-ps)

For more details, please refer to the build report.

@pvrk
Copy link
Contributor Author

pvrk commented Feb 12, 2026

@samkabue : Removed the file in a fresh commit

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants