-
Notifications
You must be signed in to change notification settings - Fork 96
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
Compliance issues with HighCharts? #300
Comments
you're not required to use high charts. you can use, for example, google charts. compliance was discussed when it was introduced. |
Ya, but they ran it through a compliance check called https://www.blackducksoftware.com/ and it failed because the highcharts code is included. Now I have no choice but to remove Metric Fu. |
Well, for one, metric_fu shouldn't be in your gemfile. I guess you can't argue with an algorithm. The code is free for use within certain constraints. There's no constraint for having it on your computer. OTOH, I'm happy to extract it out |
metric_fu is in my development/test group of my gemfile. However, I was under the impression Simplecov was requiring it - looks like I might have been wrong on that. Either way, the strict compliance rules I have to follow now won't allow me to use metric_fu at all unless highcharts is removed. I don't expect you to make a change just for me. If you have the time it would be appreciated as I'm sure others will eventually run into this issue as well at some point. |
Ya, I was digging around their licensing agreement a bit and I think you bring up a valid point. The section I want to reference is:
Here are a few other relevant links: If this is indeed the case, we might consider replacing Highcharts with another charting tool. Anyone have thoughts on the matter? I hope I'm missing something. |
Has anyone run into compliance issues with HighCharts when using metric fu?
Our app just went through a compliance audit and to my surprise metric fu (included via simplecov) threw up red flags because high charts is included in the gem and requires a commercial license.
A version of metric_fu without the high charts library would be very helpful in this case.
The text was updated successfully, but these errors were encountered: