Skip to content

CI workarounds for 16.8 Preview 3#48484

Merged
sharwell merged 8 commits intodotnet:release/dev16.8from
sharwell:enable-lookup-lite
Oct 10, 2020
Merged

CI workarounds for 16.8 Preview 3#48484
sharwell merged 8 commits intodotnet:release/dev16.8from
sharwell:enable-lookup-lite

Conversation

@sharwell
Copy link
Contributor

@sharwell sharwell commented Oct 9, 2020

No description provided.

@sharwell sharwell requested review from a team as code owners October 9, 2020 22:52
@sharwell sharwell requested a review from a team October 9, 2020 22:52
// The _owner flag is cleared on Dispose. This should only occur when cancellation has already been
// requested, but at least in 16.8 Preview 3.0 it was not always the case.
cancellationToken.ThrowIfCancellationRequested();
//Assumes.Present(provider);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just so we don't lose the request I had on the other PR: let's get a bug URL in the code pointing to this.

Comment on lines +14 to +17
var stackTrace = new StackTrace();
foreach (var frame in stackTrace.GetFrames())
{
if (frame.GetMethod()?.DeclaringType?.FullName?.Contains("ChangedText") ?? false)
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This worries me more than just commenting out the assert...

@jinujoseph jinujoseph added this to the 16.8 milestone Oct 10, 2020
@sharwell sharwell merged commit a48db44 into dotnet:release/dev16.8 Oct 10, 2020
@sharwell sharwell deleted the enable-lookup-lite branch October 10, 2020 05:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants