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 test data from published crate. #136

Merged
merged 1 commit into from
Mar 18, 2019
Merged

Exclude test data from published crate. #136

merged 1 commit into from
Mar 18, 2019

Conversation

jdm
Copy link
Member

@jdm jdm commented Mar 18, 2019

The crates.io limit is 10mb, but the latest package is 20mb. The test data is 32mb out of 43mb uncompressed, and the crate builds fine without it. The final package size is 2.2mb.

This change is Reviewable

@jdm
Copy link
Member Author

jdm commented Mar 18, 2019

r? @mbrubeck

@mbrubeck
Copy link
Contributor

@bors-servo r+

@bors-servo
Copy link
Contributor

📌 Commit f6d4c78 has been approved by mbrubeck

bors-servo pushed a commit that referenced this pull request Mar 18, 2019
Exclude test data from published crate.

The crates.io limit is 10mb, but the latest package is 20mb. The test data is 32mb out of 43mb uncompressed, and the crate builds fine without it. The final package size is 2.2mb.

<!-- 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/136)
<!-- Reviewable:end -->
@bors-servo
Copy link
Contributor

⌛ Testing commit f6d4c78 with merge 27bd7e9...

@bors-servo
Copy link
Contributor

☀️ Test successful - checks-travis
Approved by: mbrubeck
Pushing 27bd7e9 to master...

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.

None yet

3 participants