-
Notifications
You must be signed in to change notification settings - Fork 97
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
Added configuration for view precompilation. #846
Conversation
{ | ||
if (BackgroundCompilationDelay.HasValue && Mode == ViewCompilationMode.Lazy) | ||
{ | ||
throw new Exception($"{nameof(BackgroundCompilationDelay)} is not supported in {nameof(ViewCompilationMode.Lazy)} {nameof(Mode)}."); |
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.
BackgroundCompilationDelay must be null in ...
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.
We should also check that the delay is null when Mode is DuringStartup. That would only stop the startup, which does not make much sense.
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.
Fixed
@@ -93,7 +93,7 @@ public static IEnumerable<ControlUsageError> ValidateUsage(ResolvedControl contr | |||
{ | |||
if ((int)delayProperty.Value < 0) | |||
{ | |||
yield return new ControlUsageError("Delay cannot be set to negative number."); | |||
yield return new ControlUsageError("BackgroundCompilationDelay cannot be set to negative number."); |
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.
No, this is wrong 😅
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.
Ups :O
I've reverted the change and modified all the messages in UpdateProgress to use nameof
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.
Nice :)
It seems like that integration tests are failing - @MichalTichy please look at it. |
Tests seem to be ok now. |
This PR adds configuration for view compilation modes.
Compilation modes are:
Current mode of compilation.
Compilation will run during application startup. Application will start after compilation is done.
Compilation will run after application startup.