Skip to content
This repository was archived by the owner on Dec 18, 2017. It is now read-only.

More visible warning/error #1355

Closed
wants to merge 1 commit into from
Closed

More visible warning/error #1355

wants to merge 1 commit into from

Conversation

BrennanConroy
Copy link
Member

@ghost ghost added the cla-not-required label Mar 9, 2015
@muratg
Copy link
Contributor

muratg commented Mar 9, 2015

:shipit:

@@ -151,18 +151,18 @@ public Task<IEnumerable<PackageInfo>> FindPackagesByIdAsync(string id)
{
_ignored = true;
_reports.Information.WriteLine(
string.Format("Failed to retrieve information from remote source '{0}'".Yellow(),
string.Format("Failed to retrieve information from remote source '{0}'".Yellow().Bold(),
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

move this to append string.Format like the others

@BrennanConroy
Copy link
Member Author

@Praburaj

@davidfowl
Copy link
Member

:shipit:

@davidfowl
Copy link
Member

Thought you might want to test it in the VS output to see how it looks

@Praburaj
Copy link
Contributor

Looks good to me :shipit:

@BrennanConroy
Copy link
Member Author

VS output doesn't do color (if it does please tell me how!) so it is unaffected by this change.

@BrennanConroy BrennanConroy deleted the brecon/issue1051 branch March 12, 2015 16:09
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants