- 
                Notifications
    
You must be signed in to change notification settings  - Fork 10.5k
 
[blazor] startup sequence unification #46532
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
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good from my perspective.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good to me 🙂
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM 👍
Summary
This unifies the startup sequence with the runtime.
Contributes to dotnet/runtime#74346
Depends on dotnet/runtime#81886
Changes
createAPIBlazor._internalis initialized after runtime instance creationFixes
LANGenvironment based onapplicationCultureoption even if ICU is not sharded to make it work according to docsLoadingApp_KoreanLanguage_WorksShould still work as before
Out of scope
blazor.boot.jsoninto existingmono-config.jsonschema. Unification of schema will be separate work.