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

Add colours of France #245

Open
wants to merge 7 commits into
base: master
Choose a base branch
from
Open

Add colours of France #245

wants to merge 7 commits into from

Conversation

j3bb9z
Copy link

@j3bb9z j3bb9z commented Oct 9, 2018

🇫🇷

It's almost like America, but the opposite. ;)

DABH and others added 7 commits August 22, 2018 11:34
As mentioned [here](Marak#240 (comment)), there's a potential
conflict between FORCE_COLOR and FORCE_NO_COLOR. Decided to address this by issuing a warning (when supported)
and falling back on FORCE_COLOR, since that's will preserve existing behavior.
Marak#21 Add support for FORCE_NO_COLOR env var
@Marak
Copy link
Owner

Marak commented Oct 9, 2018

Thank you @jacob87o2, this is nice!

I'm wondering though, how many country flags should be included?

If we add France, wouldn't we be obligated to merge any other country flags? To my limited recollection America was added as an easter egg.

What do you think @DABH ?

@j3bb9z
Copy link
Author

j3bb9z commented Oct 9, 2018

No worries, I won't feel offended if my PR won't get through. I actually expected that.

I just couldn't help myself finding out the correlation between America and France. ;-)

Copy link

@elegantShock2258 elegantShock2258 left a comment

Choose a reason for hiding this comment

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

[removed]

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.

5 participants