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

markFeatureWriter: Use new ufo2ft MarkFeatureWriter #1030

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

khaledhosny
Copy link
Collaborator

The code here was merged back to ufo2ft’s MarkFeatureWriter. Keeping ContextualMarkFeatureWriter as stub just in case someone is using it directly. Keeping the tests too, to exercise exporting anchor libData to UFO.

@khaledhosny
Copy link
Collaborator Author

Should be merged after the new ufo2ft release and requiring it as minimum supported version.

@khaledhosny khaledhosny force-pushed the move-contextual-mark-feature-writer-to-ufo2ft branch from 6a0cf3d to 171dd6d Compare September 5, 2024 10:52
@khaledhosny khaledhosny force-pushed the move-contextual-mark-feature-writer-to-ufo2ft branch from 171dd6d to b1d41e9 Compare September 19, 2024 11:16
@khaledhosny khaledhosny marked this pull request as ready for review September 19, 2024 11:16
@khaledhosny khaledhosny force-pushed the move-contextual-mark-feature-writer-to-ufo2ft branch 3 times, most recently from 4325c2d to 6adf304 Compare September 19, 2024 11:25
The code here was merged back to ufo2ft’s MarkFeatureWriter. Keeping
ContextualMarkFeatureWriter as stub just in case someone is using it
directly. Keeping the tests too, to exercise exporting anchor libData to
UFO.
@khaledhosny khaledhosny force-pushed the move-contextual-mark-feature-writer-to-ufo2ft branch from 6adf304 to d436f8f Compare September 19, 2024 11:27
@khaledhosny khaledhosny merged commit 664779c into main Sep 19, 2024
9 of 10 checks passed
@khaledhosny khaledhosny deleted the move-contextual-mark-feature-writer-to-ufo2ft branch September 19, 2024 11:32
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.

1 participant