Add FileTypesForVersionExpiration parameter to Set-PnPTenant cmdlet

[gautamdsheth]

Before creating a pull request, make sure that you have read the contribution file located at

https://github.com/pnp/powerShell/blob/dev/CONTRIBUTING.md

Type

• Bug Fix
• New Feature
• Sample

Related Issues?

Fixes #X, partially fixes #Y, mentioned in #Z, etc.

What is in this Pull Request ?

Please describe the changes in the PR.

Guidance

• You can delete this section when you are submitting the pull request.*
• Please update this PR information accordingly. We use this as part of our release notes in monthly communications.
• Please target your PR to Dev branch. If you do not target the Dev branch we will not accept this PR.

[Copilot]

Pull request overview

This PR adds support for specifying file-type-specific version expiration overrides when running Set-PnPTenant.

Changes:

• Added a new -FileTypesForVersionExpiration parameter to the Set-PnPTenant cmdlet implementation.
• Added cmdlet logic to validate the parameter and invoke Tenant.SetFileTypeVersionPolicy(...).
• Updated Set-PnPTenant documentation to include the new parameter and usage guidance.

Reviewed changes

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

File	Description
src/Commands/Admin/SetTenant.cs	Introduces the new parameter and applies the file type version policy via CSOM.
documentation/Set-PnPTenant.md	Documents the new parameter in syntax and parameter reference.

[Copilot]

The validation for combining FileTypesForVersionExpiration with the version policy parameters relies on modified, but modified becomes true when any of ExpireVersionsAfterDays, MajorVersionLimit, or EnableAutoExpirationVersionTrim is provided (e.g., only MajorVersionLimit). This does not enforce the stated requirement of combining with EnableAutoExpirationVersionTrim OR with both ExpireVersionsAfterDays and MajorVersionLimit, and it can lead to calling SetFileTypeVersionPolicy with -1 values for missing parameters.

Consider replacing if (!modified) with an explicit check like EnableAutoExpirationVersionTrim.HasValue || (ExpireVersionsAfterDays.HasValue && MajorVersionLimit.HasValue) (and error if the requirement isn’t met).

[Copilot]

The newly added block uses tab indentation (\t) while the surrounding code in this method is indented with spaces. This introduces mixed whitespace in the same scope and can cause style/analyzer failures and noisy diffs.

Please re-indent this block to match the existing indentation used in the rest of ExecuteCmdlet() in this file.

Suggested change

	bool fileTypeVersionPolicyModified = false;
	if (FileTypesForVersionExpiration != null)
	{
	if (FileTypesForVersionExpiration.Length == 0 || FileTypesForVersionExpiration.Any(string.IsNullOrWhiteSpace))
	{
	throw new PSArgumentException($"The parameter {nameof(FileTypesForVersionExpiration)} must contain one or more non-empty file types.", nameof(FileTypesForVersionExpiration));
	}
	FileTypesForVersionExpiration = FileTypesForVersionExpiration
	.Distinct(StringComparer.OrdinalIgnoreCase)
	.ToArray();
	if (!modified)
	{
	throw new PSArgumentException($"The parameter {nameof(FileTypesForVersionExpiration)} must be combined with {nameof(EnableAutoExpirationVersionTrim)} or with both {nameof(ExpireVersionsAfterDays)} and {nameof(MajorVersionLimit)}.", nameof(FileTypesForVersionExpiration));
	}
	fileTypeVersionPolicyModified = true;
	}
	bool fileTypeVersionPolicyModified = false;
	if (FileTypesForVersionExpiration != null)
	{
	if (FileTypesForVersionExpiration.Length == 0 || FileTypesForVersionExpiration.Any(string.IsNullOrWhiteSpace))
	{
	throw new PSArgumentException($"The parameter {nameof(FileTypesForVersionExpiration)} must contain one or more non-empty file types.", nameof(FileTypesForVersionExpiration));
	}
	FileTypesForVersionExpiration = FileTypesForVersionExpiration
	.Distinct(StringComparer.OrdinalIgnoreCase)
	.ToArray();
	if (!modified)
	{
	throw new PSArgumentException($"The parameter {nameof(FileTypesForVersionExpiration)} must be combined with {nameof(EnableAutoExpirationVersionTrim)} or with both {nameof(ExpireVersionsAfterDays)} and {nameof(MajorVersionLimit)}.", nameof(FileTypesForVersionExpiration));
	}
	fileTypeVersionPolicyModified = true;
	}

[Copilot]

FileTypesForVersionExpiration values are checked with IsNullOrWhiteSpace, but they are not trimmed/normalized before de-duplication and being passed to SetFileTypeVersionPolicy. Inputs like " pdf " will pass validation and be sent with leading/trailing whitespace.

Consider trimming each entry (and then re-validating for empties) before calling Distinct(...) and applying the policy.