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

chore: use NotoSerifCJK-Regular.ttc instead of NotoSerifCJKkr-Regular.otf #242

Closed
wants to merge 4 commits into from

Conversation

da-liii
Copy link
Contributor

@da-liii da-liii commented Dec 27, 2023

  • NotoSerifCJKkr-Regular.otf is a subset of NotoSerifCJK-Regular.ttc, use ttc and remove otf to reduce the size of the git repo.
  • enable xmake ci

@galkahana
Copy link
Owner

why would i want that? i got one test for this, one test for this. im good with that.

@galkahana
Copy link
Owner

galkahana commented Dec 27, 2023

how about we create one PR for just the xmake changes, and the try to see how we got about exporting without testing in another?

i'll take care of this sometime today (and with the other thing as well - creating packages without testing)

@da-liii da-liii closed this Dec 27, 2023
galkahana added a commit that referenced this pull request Dec 27, 2023
galkahana added a commit that referenced this pull request Dec 27, 2023
@galkahana
Copy link
Owner

@darcy-shen i incorporated the changes to the xmake config you added here in:
#246

please let me know if i correctly represented your intent.
if so i'll merge...and i think it'd be proper to create a new version.

galkahana added a commit that referenced this pull request Dec 27, 2023
* conf: updates to xmake config per #242

* had dual vscode files filtering
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.

2 participants