Thanks for this. I've left some comments inline. In addition to that:
- There should probably be a build-config option for enabling the chromedriver binary (ENABLE_CHROMEDRIVER ?) - It's missing an install target.
« Back to merge proposal
Thanks for this. I've left some comments inline. In addition to that:
- There should probably be a build-config option for enabling the chromedriver binary (ENABLE_ CHROMEDRIVER ?)
- It's missing an install target.