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

Linking an actual boxplot demo #1094

Closed
wants to merge 3 commits into from
Closed

Conversation

ubershmekel
Copy link
Contributor

No description provided.

@gordonwoodhull gordonwoodhull added this to the v2.0 milestone Feb 2, 2016
@gordonwoodhull
Copy link
Contributor

Thanks @ubershmekel, looks like an errant copy-paste. The docs are generated from the source and the correct place to fix this is in src/box-plot.js; will do!

@ubershmekel
Copy link
Contributor Author

Cool. Thanks for the pointer. I updated the source. I did not take the time to regenerate the docs and test though. Feel free to close the PR if you prefer to do it yourself.

@gordonwoodhull
Copy link
Contributor

Thanks, I'll pull this in my next sweep!

@gordonwoodhull gordonwoodhull modified the milestone: v2.0 Feb 9, 2016
@gordonwoodhull
Copy link
Contributor

Merged in 2.0 beta 26. Thanks @ubershmekel!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants