Skip to content

Add Hoefler & Co test texts from https://github.com/hoeflerco/proofs#15

Draft
simoncozens wants to merge 1 commit intosile-typesetter:masterfrom
simoncozens:hoefler-texts
Draft

Add Hoefler & Co test texts from https://github.com/hoeflerco/proofs#15
simoncozens wants to merge 1 commit intosile-typesetter:masterfrom
simoncozens:hoefler-texts

Conversation

@simoncozens
Copy link
Copy Markdown
Member

Closes #14

@simoncozens simoncozens marked this pull request as draft May 19, 2020 16:31
@simoncozens
Copy link
Copy Markdown
Member Author

Hold fire - I'm going to see if there's a better way to track the hoefler repo than just pulling in the text as a once-off.

@alerque
Copy link
Copy Markdown
Member

alerque commented May 19, 2020

Why is the Lua file here marked as a binary?

I this a usecase for a git subtree? I haven't looked at how the original is updated or maintained...

@DavidLRowe
Copy link
Copy Markdown

When I look at the .lua file with Notepad++, it shows NUL characters on lines 6 and 7 just before the close quote. Those may or may not cause a problem, but they probably should be removed and that might make GitHub happy as well.

@jvgaultney
Copy link
Copy Markdown
Contributor

jvgaultney commented May 20, 2020

No subtrees please - they are just a pain and certainly not worth the bother in this case. I'd prefer a simple incorporation of the appropriate text data and his CC4 license.

@simoncozens
Copy link
Copy Markdown
Member Author

@jvgaultney
Copy link
Copy Markdown
Contributor

Hold on - I've now noticed that he's used cc-by-nc-sa, which means we can't use it. I've seen his thread on this, but whatever his intent, nc would place a restriction on fontproof. So sorry, can't do unless his drops the nc.

@alerque alerque force-pushed the master branch 5 times, most recently from c1cd785 to 3f9aeef Compare March 25, 2023 20:51
@alerque alerque force-pushed the master branch 3 times, most recently from ba3eed0 to 40cd83e Compare September 26, 2024 19:19
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.

Add Hoefler & Co proof texts

4 participants