Skip to content
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

Use CommunityToolkit converters #34936

Closed
wants to merge 176 commits into from

Conversation

krschau
Copy link

@krschau krschau commented Sep 17, 2024

No description provided.

zadjii-msft and others added 30 commits August 27, 2024 09:34
* Moving plugin folder

* all .net now pointing to the tool ver prop

* fixing cpp proj

* consolidation of packages

* removing non root nuget config

* Update src/modules/cmdpal/ToolingVersions.props

---------

Co-authored-by: Mike Griese <[email protected]>
Getting more of the projects broken into into files. Based on #3
* updating namespace for common

* Adjusting folder structure now so we can do larger move
* Adding in exts

* modifying exts

* deleting outdated azure extentions

* removing x86
As in title.

I'm adding a `ExtensionObject<T>` object too. It doesn't _enforce_ anything. We can use it to wrap up things we get from extensions. You get the object back out with `.Unsafe`, which is a mental clue "this object might not live in this process". It'll at least give us a better clue of all the places where accessing the object might not totally be safe.

Also fixes a bug that makes cmdpal a bit more resilient to an extension dying and being reloaded. Just go to all apps & back, and presto, reload.
* makes `quit` not crash the app
* makes any close actually tell our extensions to exit

once again, thanks devhome folks
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.

6 participants