-
Notifications
You must be signed in to change notification settings - Fork 4k
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
Revert change to IIncrementalAnalyzer interface #12074
Conversation
FYI @dotnet/roslyn-interactive |
<Name>Features</Name> | ||
<ReferenceOutputAssembly>false</ReferenceOutputAssembly> |
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.
It feels odd that we're reverting this. Was it part of the original change?
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.
It looks like it was a part of the original commit 1b6a83f although I'm not sure why VS updated this file since I did not modify this file 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.
Actually, I looked at the rest of this project, and we do this for many of the other references. My guess is we shouldn't have deleted this in the first place. We should just make sure we don't delete it again in features-testimpact.
LGTM (except |
Revert "Merge pull request #12074 from drognanar/revertreasons"
Reverts PR #11275