Skip to content

travis#10

Open
graingert wants to merge 7 commits intomasterfrom
travis
Open

travis#10
graingert wants to merge 7 commits intomasterfrom
travis

Conversation

@graingert
Copy link
Copy Markdown
Collaborator

No description provided.

Thomas Grainger added 2 commits July 12, 2017 21:11
@graingert
Copy link
Copy Markdown
Collaborator Author

@PeterJCLaw I've hacked the tests into passing, but at what cost?

There's a skipped test, and some of the images have changed.

Most of the images have changed how the text is rendered: pay attention to the letter 'a'

All of the azteccode and datamatrix barcodes have changed significantly, why? I don't know.

@PeterJCLaw
Copy link
Copy Markdown
Contributor

Hrm, if the visuals have changed I wonder if there's some dependent font we're missing?

@PeterJCLaw
Copy link
Copy Markdown
Contributor

It might also be interesting to rewind the history to see if there was a point when they did pass.

@graingert
Copy link
Copy Markdown
Collaborator Author

Edit, I got the tests passing locally, but it looks like it's different on Travis. Probably fonts

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants