-
Notifications
You must be signed in to change notification settings - Fork 28.7k
Auto-solito support #80968
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
base: canary
Are you sure you want to change the base?
Auto-solito support #80968
Conversation
Allow CI Workflow Run
Note: this should only be enabled once the PR is ready to go and can only be enabled by a maintainer |
@@ -196,6 +196,23 @@ async fn base_resolve_options( | |||
environment.resolve_extensions().owned().await? | |||
} else { | |||
let mut ext = Vec::new(); | |||
|
|||
if opt.enable_react_native_web_infix { |
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.
If you have opt.custom_extensions
set, then this branch is never taken. Would you have expected that behavior (that you'd have to specify .web.*
in there manually)?
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.
As of now, solito
anticipates this behavior:
However, I wonder if the custom extensions should just come first and them merge with these?
…t.js into nandorojo/auto-solito
CodSpeed Performance ReportMerging #80968 will not alter performanceComparing Summary
Benchmarks breakdown
|
No description provided.