Skip to content

Add exists option to ProductSearch Result for matching other search types#641

Merged
robmen merged 2 commits into
wixtoolset:mainfrom
metsma:ProductSearchExists
Dec 13, 2025
Merged

Add exists option to ProductSearch Result for matching other search types#641
robmen merged 2 commits into
wixtoolset:mainfrom
metsma:ProductSearchExists

Conversation

@metsma
Copy link
Copy Markdown
Contributor

@metsma metsma commented May 26, 2025

(e.g. FileSearch, DirectorySearch, RegistrySearch)

Closes wixtoolset/issues#9079

Signed-off-by: Raul Metsma raul@metsma.ee

@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 26, 2025

CLA Assistant Lite bot All contributors have signed the CLA ✍️ ✅

@metsma
Copy link
Copy Markdown
Contributor Author

metsma commented May 26, 2025

I have read the CLA Document and I hereby sign the CLA

wixbot added a commit to wixtoolset/.github that referenced this pull request May 26, 2025
@barnson
Copy link
Copy Markdown
Member

barnson commented May 26, 2025

Please open an issue so it can go through triage for approval.

@robmen robmen self-requested a review June 18, 2025 19:20
@robmen robmen self-assigned this Jun 18, 2025
Copy link
Copy Markdown
Member

@robmen robmen left a comment

Choose a reason for hiding this comment

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

This is a pretty good code change. Thanks.

I'd like to see a test added as well to make sure we don't regress this behavior.

Comment thread src/burn/engine/search.cpp
Comment thread src/burn/engine/search.cpp Outdated
@metsma metsma force-pushed the ProductSearchExists branch 2 times, most recently from a5496b4 to 17b965b Compare June 20, 2025 07:18
metsma and others added 2 commits December 13, 2025 13:20
@robmen robmen force-pushed the ProductSearchExists branch from 17b965b to d96439f Compare December 13, 2025 21:41
@robmen robmen enabled auto-merge (rebase) December 13, 2025 21:43
@robmen robmen self-requested a review December 13, 2025 21:43
@robmen robmen merged commit 7760a8b into wixtoolset:main Dec 13, 2025
2 checks passed
@github-actions github-actions Bot locked and limited conversation to collaborators Dec 13, 2025
@metsma metsma deleted the ProductSearchExists branch December 14, 2025 19:21
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add exists option to ProductSearch Result for matching other search types

3 participants