Skip to content

PowerToys Extension: Use project build because we don't need packagereference#46080

Merged
vanzue merged 1 commit intomainfrom
dev/vanzue/powertoys-extension-sdk-upgrade
Mar 16, 2026
Merged

PowerToys Extension: Use project build because we don't need packagereference#46080
vanzue merged 1 commit intomainfrom
dev/vanzue/powertoys-extension-sdk-upgrade

Conversation

@vanzue
Copy link
Contributor

@vanzue vanzue commented Mar 12, 2026

Summary of the Pull Request

At first version, tried to use the project reference instead of package reference, while it did not work in my local environment, so used package reference instead.

While planned to project reference for many reasons like first day problem explosure in sdk, maintain the strict consistent winmd with the extension host.

Hopefully to solve some of the extension not starting problem.

PR Checklist

  • Closes: #xxx
  • Communication: I've discussed this with core contributors already. If the work hasn't been agreed, this work might be rejected
  • Tests: Added/updated and all pass
  • Localization: All end-user-facing strings can be localized
  • Dev docs: Added/updated
  • New binaries: Added on the required places
  • Documentation updated: If checked, please file a pull request on our docs repo and link it here: #xxx

Detailed Description of the Pull Request / Additional comments

Validation Steps Performed

Built an installer and validated in my arm64 and x64 machine, works perfectly

Copy link
Collaborator

@jiripolasek jiripolasek left a comment

Choose a reason for hiding this comment

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

LGTM. I have no issue deploying this locally... and that scares me. I'm going to circle back and figure out why I have a problem deploying it with the Extensions SDK from NuGet.org.

As a minor detail, I took the liberty of updating the SLNF for the extension: #46136 - but that just devexp.

Image

@vanzue
Copy link
Contributor Author

vanzue commented Mar 16, 2026

Thanks!!

@vanzue vanzue merged commit 0c2d24c into main Mar 16, 2026
22 checks passed
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.

2 participants