Skip to content
This repository has been archived by the owner on Jul 15, 2023. It is now read-only.

Disable the delve --log option by default. #352

Merged
merged 2 commits into from
Jun 7, 2016
Merged

Conversation

bawr
Copy link
Contributor

@bawr bawr commented Jun 6, 2016

This is a small patch that makes it possible to toggle delve's debugging output.

I've found it to introduce a lot of pointless noise, distracting from meaningful stderr output.

bawr added 2 commits June 6, 2016 16:15
TODO: Are the typings correct? Should we add an empty .body instead?
@msftclas
Copy link

msftclas commented Jun 6, 2016

Hi @bawr, I'm your friendly neighborhood Microsoft Pull Request Bot (You can call me MSBOT). Thanks for your contribution!

In order for us to evaluate and accept your PR, we ask that you sign a contribution license agreement. It's all electronic and will take just minutes. I promise there's no faxing. https://cla.microsoft.com.

TTYL, MSBOT;

@msftclas
Copy link

msftclas commented Jun 6, 2016

@bawr, Thanks for signing the contribution license agreement so quickly! Actual humans will now validate the agreement and then evaluate the PR.

Thanks, MSBOT;

@lukehoban
Copy link
Contributor

Looks good - thanks!

@lukehoban lukehoban merged commit 9abf0fb into microsoft:master Jun 7, 2016
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants