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 promotions between Gray and multi-component Colorants #272

Merged
merged 2 commits into from Feb 7, 2017

Conversation

timholy
Copy link
Member

@timholy timholy commented Feb 4, 2017

Amazing that we haven't noticed the lack of these previously. These rules aren't perfect, but I'll file an issue detailing what needs to improve.

@tkelman, I also noticed some stale (>1 year old) @deprecate statements, which I deleted. Does deleting those mean we need to do a minor version bump?

These have been present since 2015-08-14; more than 1 year seems like plenty of time
@tkelman
Copy link

tkelman commented Feb 4, 2017

That would be the more cautious option, yeah. Any idea if people had been using the old names in packages that weren't updated?

@timholy
Copy link
Member Author

timholy commented Feb 4, 2017

I have 26 of the 52 Colors dependents installed. grepping all the *.jl files for rgb( did not reveal anything besides a lot of false positives. Good enough for me.

@timholy timholy changed the title Add promotions between Gray and AbstractRGB Add promotions between Gray and multi-component Colorants Feb 4, 2017
@timholy timholy merged commit ea7aa16 into master Feb 7, 2017
@timholy timholy deleted the teh/promotion branch February 7, 2017 12:12
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.

None yet

2 participants