Skip to content

Add Microsoft.WSL profile source to schema #19047

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

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

chawyehsu
Copy link

@chawyehsu chawyehsu commented Jun 17, 2025

Summary of the Pull Request

WSL now generates profiles with the source named Microsoft.WSL, this PR adds the value to the profile schema.

References and Relevant Issues

#18231

Copy link
Member

@carlos-zamora carlos-zamora left a comment

Choose a reason for hiding this comment

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

Thanks!

@carlos-zamora
Copy link
Member

Extracted from PR body:
image

@carlos-zamora carlos-zamora enabled auto-merge (squash) June 18, 2025 22:58
@carlos-zamora
Copy link
Member

@DHowett you may have to admin merge since this PR only touches the schema

@DHowett
Copy link
Member

DHowett commented Jun 19, 2025

Considering that Fragments are allowed to have any name (like Microsoft.WSL or Foo.Bar), we may want to just lift the constraint entirely!

@DHowett
Copy link
Member

DHowett commented Jun 19, 2025

(ergo, change it from an enum to a plain string in the schema)

Copy link
Member

@DHowett DHowett left a comment

Choose a reason for hiding this comment

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

as noted

@microsoft-github-policy-service microsoft-github-policy-service bot added the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Jun 19, 2025
auto-merge was automatically disabled June 19, 2025 03:19

Head branch was pushed to by a user without write access

@microsoft-github-policy-service microsoft-github-policy-service bot removed the Needs-Author-Feedback The original author of the issue/PR needs to come back and respond to something label Jun 19, 2025
@chawyehsu
Copy link
Author

@microsoft-github-policy-service agree

@chawyehsu
Copy link
Author

we may want to just lift the constraint entirely!

What about now? I've relaxed the constraint to accept any string but leave the enum to keep supporting tab-completion for known variants.

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.

4 participants