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

New awesome image comparison using pixelmatch #50

Merged
merged 1 commit into from
Oct 16, 2015
Merged

Conversation

mourner
Copy link
Member

@mourner mourner commented Oct 16, 2015

Makes the test suite many times more awesome and useful by ditching ImageMagick in favor of my new pixelmatch library. Closes #24.

Take a look at the example output and compare that with the current one.

cc @mapbox/gl

@friedbunny
Copy link

I need to up my game so I can use "awesome" in my PRs, too. Rock on, @mourner. 🎸

@jfirebaugh jfirebaugh merged commit ce801ba into master Oct 16, 2015
@jfirebaugh jfirebaugh deleted the pixelmatch branch October 16, 2015 19:34
@jfirebaugh
Copy link
Contributor

possibly update outdated reference images

Checked this, the reference image for geojson test does not visually change when I update it, so the difference in CI is something else.

@mourner
Copy link
Member Author

mourner commented Oct 16, 2015

@friedbunny just add the word "awesome" to your every PR, it's a daily autosuggestion ritual :)

friedbunny added a commit to mapbox/mapbox-gl-native that referenced this pull request Oct 28, 2015
Drops mason-installed packages Boost, zlib, libpng, libuv and glfw.
Adds libtool and automake, which may be needed when mason binaries
are not available.

Drops ImageMagick, which was replaced with pixelmatch in
aaf090b &
mapbox/mapbox-gl-test-suite#50
AndwareSsj pushed a commit to AndwareSsj/mapbox-gl-native that referenced this pull request Nov 6, 2015
Drops mason-installed packages Boost, zlib, libpng, libuv and glfw.
Adds libtool and automake, which may be needed when mason binaries
are not available.

Drops ImageMagick, which was replaced with pixelmatch in
aaf090b &
mapbox/mapbox-gl-test-suite#50
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants