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

Add HarfBuzz example #3237

Merged
merged 8 commits into from
Apr 14, 2023
Merged

Add HarfBuzz example #3237

merged 8 commits into from
Apr 14, 2023

Conversation

sffc
Copy link
Member

@sffc sffc commented Mar 31, 2023

This example runs and passes the (very basic) test.

If servo/rust-harfbuzz#197 lands, I'd like to migrate over to using that instead.

@echeran

@sffc sffc requested a review from echeran March 31, 2023 08:24
@sffc sffc added the waiting-on-author PRs waiting for action from the author for >7 days label Apr 13, 2023
@sffc sffc marked this pull request as ready for review April 13, 2023 23:15
@sffc sffc requested review from hsivonen and a team as code owners April 13, 2023 23:15
@sffc sffc removed the waiting-on-author PRs waiting for action from the author for >7 days label Apr 13, 2023
@sffc sffc requested a review from Manishearth April 13, 2023 23:26
Manishearth
Manishearth previously approved these changes Apr 13, 2023
@codecov-commenter
Copy link

Codecov Report

Merging #3237 (6e7696f) into main (0e27b4c) will increase coverage by 0.11%.
The diff coverage is n/a.

❗ Current head 6e7696f differs from pull request most recent head a8ebd71. Consider uploading reports for the commit a8ebd71 to get more accurate results

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

@@            Coverage Diff             @@
##             main    #3237      +/-   ##
==========================================
+ Coverage   74.86%   74.97%   +0.11%     
==========================================
  Files         689      689              
  Lines       43272    43272              
==========================================
+ Hits        32395    32445      +50     
+ Misses      10877    10827      -50     
Impacted Files Coverage Δ
experimental/harfbuzz/src/lib.rs 46.34% <ø> (+45.12%) ⬆️

... and 3 files with indirect coverage changes

📣 We’re building smart automated test selection to slash your CI/CD build times. Learn more

@sffc sffc merged commit 59e370f into unicode-org:main Apr 14, 2023
@sffc sffc deleted the harfbuzz-xmp branch April 14, 2023 00:49
This pull request was closed.
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.

3 participants