Check for path==None

[tpvasconcelos]

Closes #226

[ayjayt]

Hi! This was already fixed and merged by #224

• the real issue here is that they don't have chrome or chromium installed! But that fix explicitly checks for that and then explicitly raises an error.

Need to push a minor revision! Thanks!

@gvwilson @tpvasconcelos

[tpvasconcelos]

@ayjayt Thanks a lot for the feedback!

Just a question: wouldn't this warrant a major version bump (or at least minor) since it's technically a breaking change? This will break all running instances that do not have chromium installed, which will be most servers and CI systems.

Thanks again!

[ayjayt]

Yeah thats what we're considering. You really can't treat it like a pure python dependency change because the managers don't pull Chrome automatically. I think tomorrow they'll yank v0.4.x and release it as v1.0.0. i apologize for the inconvenience, I'm not a plotly employee but I am the lead developer on the project and I should have anticipated this.

(Yank won't delete the version, just make it not automatically downloaded)

[tpvasconcelos]

Thanks for clarifying and no worries at all from my side!