search

shadowmoose / pyderman

Install Selenium-compatible Chrome/Firefox/Opera/PhantomJS/Edge webdrivers automatically.
MIT License
29 stars 11 forks source link

Define errors and replace `Exception` with them #18

Open eggplants opened 2 years ago

eggplants commented 2 years ago

This package often uses Exception class when raising an error. But I suggest to replace them with more precise, detail, and internal Error classes like pyderman.Error.BrowserError to make easy to handle properly.

shadowmoose commented 2 years ago

Good idea. Generally any error raised signifies that this library has failed, but it would certainly be better to add more context there.