Skip to content

Conversation

@alexandair
Copy link
Collaborator

Copy link
Contributor

Copilot AI left a 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 pull request adds a new assessment test (Test-Assessment.35006) for validating PDF labeling support in SharePoint Online environments. The test checks whether the EnableSensitivityLabelforPDF setting is enabled in SharePoint tenant configuration.

Key changes:

  • New assessment function that queries SharePoint tenant settings for PDF labeling support
  • Documentation describing the feature and remediation steps
  • Unit tests covering pass, fail, and error scenarios

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.

File Description
src/powershell/tests/Test-Assessment.35006.ps1 Implements the main assessment logic to check if PDF labeling is enabled in SharePoint Online
src/powershell/tests/Test-Assessment.35006.md Provides documentation and remediation steps for enabling PDF labeling support
code-tests/test-assessments/Test-Assessment.35006.Tests.ps1 Contains unit tests for the assessment function covering enabled, disabled, and error scenarios

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Add a null check ensuring that `$spoTenant` cannot be null

Co-authored-by: Copilot <[email protected]>
Copilot AI review requested due to automatic review settings January 4, 2026 09:50
Information --> information

Co-authored-by: Copilot <[email protected]>
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 3 comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copilot AI review requested due to automatic review settings January 4, 2026 10:09
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

for --> For

Co-authored-by: Copilot <[email protected]>
Copilot AI review requested due to automatic review settings January 4, 2026 10:28
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Copilot reviewed 3 out of 3 changed files in this pull request and generated no new comments.


💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Collaborator

@SagarSathe SagarSathe left a comment

Choose a reason for hiding this comment

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

LGTM

@SagarSathe SagarSathe merged commit eb41d25 into microsoft:main Jan 6, 2026
8 checks passed
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