Add legend to colors in upgrade-iteractive

[bestander]

Do you want to request a feature or report a bug?
improvement

What is the current behavior?

Would be nice to explain what read and yellow colors mean

[mohitgarg]

I would like to help with this issue.

[bestander]

Awesome! Send a PR and cc me

[mohitgarg]

I am a beginner in OSS. I understood the issue but can you walk me through the files and point me in the right direction. I would really appreciate it!

[bestander]

Sure, are you comfortable making changes to Yarn source code, building it and running any command?
The code for upgrade-interactive is here https://github.com/yarnpkg/yarn/blob/master/src/cli/commands/upgrade-interactive.js

[clarenced]

Hi @bestander, can I contribute to this issue?

[bestander]

You don't need to ask for a permission:) Just send a PR

…

On Wed, 31 May 2017 at 10:04, Charles Dimitri ***@***.***> wrote: Hi @bestander <https://github.com/bestander>, can I contribute to this issue? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3454 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACBdWGvjXqnDUmJUgZuWsvFcqmTtUoNmks5r_S0FgaJpZM4NgS8l> .

[clarenced]

Hi @bestander , do you have any idea what message to display for explaining the legend colors? My correction will put a key in src/reporters/lang/en.js to display the content of the message.

[bestander]

I don't have an opinion here :) Use your best judgement and let's see how it looks

…

On 1 June 2017 at 10:57, Charles Dimitri ***@***.***> wrote: Hi @bestander <https://github.com/bestander> , do you have any idea what message to display for explaining the legend colors? My correction will put a key in src/reporters/lang/en.js to displa the content of the message. — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#3454 (comment)>, or mute the thread <https://github.com/notifications/unsubscribe-auth/ACBdWK0TdkwOZam_icXBNaHK9mloAsqcks5r_osNgaJpZM4NgS8l> .

[clarenced]

Hi @bestander, here is the PR #3561

[pichouk]

Just read this issue and seems it can be closed since PR where merged ;)

[bestander]

Thanks for the ping