-
Notifications
You must be signed in to change notification settings - Fork 10.4k
Merge release/2.1 #8448
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
Merge release/2.1 #8448
Conversation
* Add some additional logging to ErrorPageMiddlewareWebSite DeveloperExceptionMiddleware will log an error if rendering the exception page throws. The test failure in https://github.com/aspnet/AspNetCore-Internal/issues/1730 suggests that we encountered an error like so but do not have anything further to go by. This change adds logging to the test so we could identify possible issues
* Restore testassets during build to prevent timeouts Fixes https://github.com/aspnet/AspNetCore-Internal/issues/1695
Possible fix to #7313 One of the characteristics of these failures were that the test took long to run. The build log has warnings for several long running tests. This might be a result of CPU contention since mondo-ification that make MVC's functional tests run with nearly every other test project in the solution
- shipped in the past patch
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.
Please hold off on this until I get the queue changes into release/2.2
. You're backing those changes out while making it seem as if they're already in this branch.
I'll be updating #7920 to install Chrome and that one working after lunch.
499cc96
to
47698fe
Compare
47698fe
to
7557cb9
Compare
:update: |
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 fine to me
No description provided.