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

fix: missing flags in Makefile and compatible changes to workflow #19

Merged
merged 11 commits into from
Jan 8, 2023
Merged

fix: missing flags in Makefile and compatible changes to workflow #19

merged 11 commits into from
Jan 8, 2023

Conversation

CookiePieWw
Copy link
Contributor

No description provided.

@skyleaworlder
Copy link
Member

非常感谢!@CookiePieWw

既然进行了这方面的修改,我还有以下提议以完善本 PR:

  • 可否以 -xelatex 作为 ENGINE 的默认选项。
  • 修改 README.md 中有关 make 的部分,毕竟 ENGINE 的指定只在 Makefile 中,个人认为有必要增加如何指定 ENGINE 的说明。
  • CI 原本都是使用 latexmk 统一编译三系统 pdf,目的之一是希望通过 test.yml 的指令让懂得一些 GitHub Actions 的同学能够一眼看出项目构建方式。(虽然我觉得不太会有人这么做就是了)目前转为使用 Makefile,我觉得可以把 windows 平台上的 latexmk 也换成使用 make.bat 的方式。

以上三点我认为可以考虑。直到 @CookiePieWw 表达对上述提议的看法前,本 PR 暂不合入。

@CookiePieWw
Copy link
Contributor Author

感谢 @skyleaworlder 的 review。

对 makefile 和 README.md 作了相应修改。对于 make.bat,由于我这里没有 win 的编译环境,因此只能尝试修改并使用 workflow 检查,不知道是什么原因,目前还无法成功生成 pdf 并上传。您对此有什么看法呢?

Copy link
Member

@skyleaworlder skyleaworlder left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

或许应该是这里的问题,由于目前我也不是很方便,所以也很难测试。

不断尝试修复 CI 并使之通过的过程是非常痛苦的,我非常理解,真是麻烦同学了!

make.bat Show resolved Hide resolved
@skyleaworlder skyleaworlder merged commit 040c91d into TJ-CSCCG:master Jan 8, 2023
@skyleaworlder
Copy link
Member

再次感谢 @CookiePieWw

修复 CI 在 GitHub Actions 上始终是一件痛苦的事情,感谢你对 tongji-undergrad-thesis 在多平台适配问题上所做出的努力。

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

2 participants