Skip to content

Filter out missing import fixes that would add references#36251

Merged
JoeRobich merged 2 commits intodotnet:masterfrom
JoeRobich:do-not-add-references
Jun 11, 2019
Merged

Filter out missing import fixes that would add references#36251
JoeRobich merged 2 commits intodotnet:masterfrom
JoeRobich:do-not-add-references

Conversation

@JoeRobich
Copy link
Copy Markdown
Member

@JoeRobich JoeRobich commented Jun 7, 2019

Allow the user to choose when they want to add and which references they want to add to their project.

Resolves #34398, since the error is triggered by a foreground assertion when adding Framework assembly references.

@JoeRobich JoeRobich requested a review from a team as a code owner June 7, 2019 22:59
@JoeRobich JoeRobich changed the title Filter out missing import fixes that would add references WIP: Filter out missing import fixes that would add references Jun 7, 2019
@JoeRobich JoeRobich force-pushed the do-not-add-references branch from bb79480 to 6697d74 Compare June 8, 2019 18:10
@JoeRobich JoeRobich changed the title WIP: Filter out missing import fixes that would add references Filter out missing import fixes that would add references Jun 10, 2019
@JoeRobich
Copy link
Copy Markdown
Member Author

@sharwell please take a look

@JoeRobich
Copy link
Copy Markdown
Member Author

@vatsalyaagrawal for approval

@jinujoseph jinujoseph added this to the 16.2 milestone Jun 11, 2019
@JoeRobich JoeRobich merged commit 9948e3d into dotnet:master Jun 11, 2019
@JoeRobich JoeRobich deleted the do-not-add-references branch February 7, 2020 18:29
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.

CSharpAddMissingImportsRefactoringProvider encountered an error

4 participants