Maciej Brencz
Maciej Brencz
@gmetais, thanks for the investigation. I"ll try to refactor the test to be more deterministic and work across platforms.
Great idea! There is already a JSON file with metrics metadata, but adding `importance` field for each metric would be definitely a killer feature.
I'm totally for this idea, no irony intended :) However, I see it more as a long lasting "project". Marking 100+ metrics with a propery "priority" / "importance grade" will...
Good idea! - First, you can take a look at a `userAgent` extension on how to handle options and customise puppeteer run. https://github.com/macbre/phantomas/blob/devel/extensions/userAgent/userAgent.js - And use puppeteer API to set...
@rbayliss, if you feel like preparing a PR, that would be great :)
Waiting for https://github.com/ariya/phantomjs/pull/404
Tools to consider: - [cwebp](https://developers.google.com/speed/webp/docs/cwebp) - [mozjpeg](https://github.com/mozilla/mozjpeg/blob/master/usage.txt) - [pngcrush](http://pmt.sourceforge.net/pngcrush/)
@collerek - you're right, this does look more like an enhancement then a bug.
@sumartoyo, thanks for your PR. Please add a test case(s) that cover your changes. Then we'll review your pull request. ``` FAIL Required test coverage of 100.0% not reached. Total...
@sumartoyo, `black` formatting tool is complaining. And linting does fail. Please fix these and we'll review your changes. Thanks!