Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

Fix issue with incomplete results when completing paths with wildcards in non-filesystem providers#24757

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
iSazonov merged 5 commits intoPowerShell:masterfromMartinGC94:FixEarlyReturn
Feb 25, 2025

Conversation

MartinGC94
Copy link
Contributor

PR Summary

Replaces an early return with a continue so the completion code continues to evaluate other resolved paths when encountering an empty path.
This resolves scenarios like:

$keys = New-Item -Path @(                'HKCU:\AB1'                'HKCU:\AB2'                'HKCU:\AB2\Test'            )$res = TabExpansion2 -inputScript 'Get-ChildItem -Path HKCU:\AB?\'

where you wouldn't getHKCU:\AB2\Test in the results because AB1 is empty.

PR Context

Fixes:#24756

PR Checklist

0xfeeddeadbeef reacted with thumbs up emoji
@iSazonov

This comment was marked as outdated.

@azure-pipelinesAzure Pipelines

This comment was marked as outdated.

@vexx32vexx32 added WG-Engine-Providersbuilt-in PowerShell providers such as FileSystem, Certificates, Registry, etc. WG-NeedsReviewNeeds a review by the labeled Working Group labelsJan 9, 2025
@iSazonov

This comment was marked as outdated.

@azure-pipelinesAzure Pipelines

This comment was marked as outdated.

@azure-pipelinesAzure Pipelines

This comment was marked as outdated.

@azure-pipelinesAzure Pipelines

This comment was marked as outdated.

@iSazonov

This comment was marked as outdated.

@azure-pipelinesAzure Pipelines

This comment was marked as outdated.

@azure-pipelinesAzure Pipelines

This comment was marked as outdated.

@microsoft-github-policy-servicemicrosoft-github-policy-servicebot added the Review - NeededThe PR is being reviewed labelJan 18, 2025
@iSazonov

This comment was marked as outdated.

@azure-pipelinesAzure Pipelines

This comment was marked as outdated.

@microsoft-github-policy-servicemicrosoft-github-policy-servicebot removed the Review - NeededThe PR is being reviewed labelFeb 19, 2025
@iSazonov
Copy link
Collaborator

/azp run

@azure-pipelinesAzure Pipelines
Copy link

Azure Pipelines successfully started running 2 pipeline(s).

@iSazonoviSazonov added the CL-GeneralIndicates that a PR should be marked as a general cmdlet change in the Change Log labelFeb 20, 2025
@iSazonoviSazonov self-assigned thisFeb 20, 2025
@iSazonov
Copy link
Collaborator

@vexx32 Is anything you want to discuss with WG or we can merge?

@vexx32
Copy link
Collaborator

As long as you're happy with the tests and behaviour after the fix, I think this is okay to merge, myself. Just wanted to make sure it got a review.

iSazonov reacted with eyes emoji

@iSazonoviSazonov removed the WG-Engine-Providersbuilt-in PowerShell providers such as FileSystem, Certificates, Registry, etc. labelFeb 25, 2025
@iSazonoviSazonov removed the WG-NeedsReviewNeeds a review by the labeled Working Group labelFeb 25, 2025
@iSazonoviSazonov merged commita23e30b intoPowerShell:masterFeb 25, 2025
39 of 41 checks passed
@microsoft-github-policy-serviceMicrosoft GitHub Policy Service
Copy link
Contributor

microsoft-github-policy-servicebot commentedFeb 25, 2025
edited by unfurl-linksbot
Loading

📣 Hey@MartinGC94, how did we do? We would love to hear your feedback with the link below! 🗣️

🔗https://aka.ms/PSRepoFeedback

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

@iSazonoviSazonoviSazonov approved these changes

Assignees

@iSazonoviSazonov

Labels
CL-GeneralIndicates that a PR should be marked as a general cmdlet change in the Change Log
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Wildcard completions in non-filesystem providers return incomplete results
3 participants
@MartinGC94@iSazonov@vexx32

[8]ページ先頭

©2009-2025 Movatter.jp