Skip to content
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

OutputConsoleLogger supports not forcing focus onto the Error List #6241

Draft
wants to merge 1 commit into
base: dev
Choose a base branch
from

Conversation

donnie-msft
Copy link
Contributor

Bug

Fixes:

Description

Draft - for an upcoming Quality Week I want to investigate if I can improve 2 things:

  1. When "Clear NuGet Local resources" completes, the Output pane is concluded by calling End() which shows "---finished---" and similar text. Unfortunately, it also explicitly tells the ErrorList to appear, which doesn't make sense. This PR fixes that, but there could be a better re-design of how this all works and that's worth thinking through.

  2. I'm not able to reproduce Package Manager Console switches to the Error List window automatically Home#6408 but need to spend more time with a real-world EF6 and EFCore scenario to ensure that. I'm wondering if there's a fix that can help Package Manager Console (PMC) as well.

PR Checklist

  • Meaningful title, helpful description and a linked NuGet/Home issue
  • Added tests
  • Link to an issue or pull request to update docs if this PR changes settings, environment variables, new feature, etc.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant