Skip to content

Add diag switch #1037

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

Merged
merged 3 commits into from
Nov 25, 2016
Merged

Add diag switch #1037

merged 3 commits into from
Nov 25, 2016

Conversation

JakeGinnivan
Copy link
Contributor

No description provided.

@pascalberger
Copy link
Member

LGTM. Maybe it would make sense to call /diag in the VSTS task if system.verbose variable is set (at least if it runs under the new agent where Git should be available)

@asbjornu
Copy link
Member

Perhaps something similar can be done in GitVersionTask?

@JakeGinnivan JakeGinnivan added this to the 4.0.0 milestone Oct 2, 2016
@JakeGinnivan
Copy link
Contributor Author

JakeGinnivan commented Nov 19, 2016

@pascalberger @asbjornu @gep13 am unsure about how to add this into GitVersionTask. I think having the CLI to diagnose issues is ok?

@gep13
Copy link
Member

gep13 commented Nov 19, 2016

@JakeGinnivan agreed

@asbjornu
Copy link
Member

Hm. Since Visual Studio has a setting for MSBuild verbosity and MSBuild itself has the /verbosity:level switch, you'd think this was something that could be detected somehow. Propagating that into GitVersionTask would be the ideal thing to do, imho.

@JakeGinnivan JakeGinnivan merged commit 9eb8711 into GitTools:master Nov 25, 2016
@JakeGinnivan JakeGinnivan deleted the AddDiagSwitch branch November 25, 2016 04:09
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.

4 participants