-
Notifications
You must be signed in to change notification settings - Fork 2.1k
Add last grey/gray spelling combination #4407
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
Conversation
Why don't we just remove |
Counter-argument: I can never remember whether it's grey or gray and use them interchangingly. |
I too can't remember it, but always use tab completion so it's better if there are fewer candidates. (But I have no strong opinion here) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'm approving this but don't have a strong opinion one way or another.
@hadley, it was recently added because it was lacking and US spelling users requested it |
@hadley
|
@yutannihilation that's just my standard response. I wouldn't read anything into it. I think reverting #4299 is the simplest fix — |
I see... Yeah, I personally agree with reverting it. |
Fine with reverting also. Maybe I'll finally learn the correct spelling. :-) |
Its A for America (grAy) and E for England (grEy) 😄 |
And what do they use in Australia? |
ok, I'll revert instead |
This is not really the best, and I'm open to alternatives.
ggplot2 originally had
scale_colour_grey()
, andscale_color_grey()
, with the former mixing up English/American spelling of colour and grey. A recent PR addedscale_color_gray()
which makes the tests fail as it looks for whether all English colour scales have an American equivalent. This PR addsscale_colour_gray()
which fixes the test, but leaves a sour taste in my mouth. Dream scenario would be to only havescale_colour_grey()
andscale_color_gray()
as those names would be internally consistent, but I don't think we can remove the original mixed one as it would cause major breakage...The alternative is to change the tests to remove one of the versions but that seems almost just as bad...
Thoughts