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

Rename #10

Closed
wants to merge 4 commits into from
Closed

Rename #10

wants to merge 4 commits into from

Conversation

EricForgy
Copy link
Contributor

I'm still learning GitHub but throwing this to you hoping it saves you some time.

Closes #9

@sglyon
Copy link
Member

sglyon commented Dec 22, 2015

Thanks! merged with 412f180

@sglyon sglyon closed this Dec 22, 2015
@sglyon sglyon mentioned this pull request Dec 22, 2015
@EricForgy
Copy link
Contributor Author

I notice Git on Windows sometimes has upper/lower case issues. Locally, I changed the file Plotlyjs.jl to PlotlyJS.jl, but it looks like it is still lowercase JS. Not a big deal I think, but maybe good to try to fix it

@EricForgy
Copy link
Contributor Author

PS: I did not test it :D

@EricForgy
Copy link
Contributor Author

All examples work and it looks great :D 👍

@sglyon
Copy link
Member

sglyon commented Dec 22, 2015

Awesome, thanks for reporting. They also worked for me so that's a good thing!

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.

2 participants