-
Notifications
You must be signed in to change notification settings - Fork 7
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
Proposal: rename this package to ember-cli-d3-shim #22
Comments
There are probably some private projects that use this :) Also, there's also a d3-shim (https://github.com/brzpegasus/ember-d3). Do you see a compelling reason to make another library? |
No, I'd like to either combine this with So to help reduce confusion about what this does, I'd like to get rid of the |
I'm sure @mingcodes would have no objection to using a common shim in |
I'm really excited by all the advances made in charting and Ember. I like the idea of renaming this package. We are currently moving away from the ember-charts library to more customized charts and the Ember addon d3 space is confusing as to what each package actually does. |
@mrosenberg we're working towards making the ecosystem more consistent. It looks like this library will be merged in to |
Considering this now just wraps
[email protected]
, I think it's time to deprecateember-cli-d3-shape
and release[email protected]
.This would require notifying everyone who is using this as a dependency, which thankfully is only 4 projects
The text was updated successfully, but these errors were encountered: