Skip to content

RestrictedCapabilities & Capabilities Validator fixes#291

Merged
yao-msft merged 5 commits into
microsoft:mainfrom
luke-patchblox:main
Jun 5, 2025
Merged

RestrictedCapabilities & Capabilities Validator fixes#291
yao-msft merged 5 commits into
microsoft:mainfrom
luke-patchblox:main

Conversation

@luke-patchblox

Copy link
Copy Markdown
Contributor

Fixes issue #290

@yao-msft

yao-msft commented Jun 4, 2025

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@yao-msft

yao-msft commented Jun 4, 2025

Copy link
Copy Markdown
Contributor

@luke-patchblox
Thanks for the fix. Can you add the "using Microsoft.WinGet.RestSource.Utils.Validators.StringValidators;" to fix the build errors?

@luke-patchblox luke-patchblox left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Added the using statements that I should not have forgotten

@yao-msft

yao-msft commented Jun 5, 2025

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@luke-patchblox

Copy link
Copy Markdown
Contributor Author

Style cop was throwing an error with the spacing which is what caused the second file's error. I think I accidentally included white space when adding in the using statements in RestrictedCapabilties.

@yao-msft

yao-msft commented Jun 5, 2025

Copy link
Copy Markdown
Contributor

/azp run

@azure-pipelines

Copy link
Copy Markdown
Azure Pipelines successfully started running 1 pipeline(s).

@yao-msft yao-msft merged commit 1208d21 into microsoft:main Jun 5, 2025
4 checks passed
@yao-msft yao-msft linked an issue Jun 5, 2025 that may be closed by this pull request
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.

Capabilities & RestrictedCapabilities model validator bug

2 participants