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

Exclude only test font data #144

Merged
merged 1 commit into from Apr 8, 2019
Merged

Exclude only test font data #144

merged 1 commit into from Apr 8, 2019

Conversation

@spl
Copy link
Contributor

spl commented Apr 8, 2019

This is a key part taken from #138 and #142. I'm creating a new PR here for just this since those others have been combined with other, somewhat orthogonal issues. I think it's best to go ahead and merge this and make a release, since this issue is affecting others. I'll continue working on the other issues.


This change is Reviewable

Fixes #137
@spl spl force-pushed the spl:fix-test-excludes branch from 7234ca7 to 9b6339b Apr 8, 2019
@jdm
Copy link
Member

jdm commented Apr 8, 2019

Agreed. Thanks!
@bors-servo r+

@bors-servo
Copy link
Contributor

bors-servo commented Apr 8, 2019

📌 Commit 9b6339b has been approved by jdm

@bors-servo
Copy link
Contributor

bors-servo commented Apr 8, 2019

Testing commit 9b6339b with merge 68a5ac1...

bors-servo added a commit that referenced this pull request Apr 8, 2019
Exclude only test font data

This is a key part taken from #138 and #142. I'm creating a new PR here for just this since those others have been combined with other, somewhat orthogonal issues. I think it's best to go ahead and merge this and make a release, since this issue is affecting others. I'll continue working on the other issues.

<!-- Reviewable:start -->
---
This change is [<img src="https://reviewable.io/review_button.svg" height="34" align="absmiddle" alt="Reviewable"/>](https://reviewable.io/reviews/servo/rust-harfbuzz/144)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

bors-servo commented Apr 8, 2019

☀️ Test successful - checks-travis
Approved by: jdm
Pushing 68a5ac1 to master...

@bors-servo bors-servo merged commit 9b6339b into servo:master Apr 8, 2019
3 checks passed
3 checks passed
Travis CI - Pull Request Build Passed
Details
continuous-integration/appveyor/pr AppVeyor build succeeded
Details
homu Test successful
Details
@spl spl deleted the spl:fix-test-excludes branch Apr 9, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

3 participants
You can’t perform that action at this time.