progit2-zh icon indicating copy to clipboard operation
progit2-zh copied to clipboard

Warning but not quit after URL check

Open networm opened this issue 1 year ago • 3 comments

Our action build failed after url check 遵循原文 修改ruby脚本过时的代码 · progit/progit2-zh@55231ea

But english repo has no problem Merge pull request #1970 from z-hed/notepad++ · progit/progit2@b3af68c

It seems that url check only output warnings, but not quit

HTML-Proofer found 8 failures!
Command failed with status (1): [htmlproofer progit.html]
Error when checking books (ignored)

Could you help us to just ignore this check? @jnavila Maybe it could be like this: Fix htmlproofer invocation · progit/progit2@8de1de7

networm avatar Aug 13 '24 16:08 networm

I submitted a PR #520 to solve the building failure problem. Here is how I resolved it:

  • First,from the building log progit2-zh/actions/runs/10380612436/job/28740723111, I found that Gemfile from https://raw.githubusercontent.com/progit/progit2-pub/master/Gemfile.new, and Rakefile from https://raw.githubusercontent.com/progit/progit2-pub/master/Rakefile.
  • I tried to use these two files, build locally, but it does not work. The ones that confused me are ENV['GITHUB_REPOSITORY'] and `ENV['TRAVIS_REPO_SLUG']'. I do not know where they are defined.
  • So why not just use the Gemfile and Rakefile files from the progit2 repository? So after a series of revisions, I made it.

Things to be attentive:

  • Ruby version: 2.7.8, It is same with the version shown in the log /opt/hostedtoolcache/Ruby/2.7.8/x64/bin.
  • The asciidoctor-pdf-cjk-kai_gen_gothic code was outdated and no longer accepted any PR, so I modified it and replaced it with my fork project address https://github.com/chloerei/asciidoctor-pdf-cjk-kai_gen_gothic/pull/16.
  • I noticed the kendlegen download from this https://web.archive.org/web/20150803131026/https://kindlegen.s3.amazonaws.com/kindlegen_linux_2.6_i3 86_v2_9.tar.gz, I couldn't download kindlegen due to some network problems, so I didn't add buid_mobi when I built it locally. But I added mobi support to the code I submitted.
  • This file (.github/workflows/pr-build.yml) had adjusted to get the new Gemfile and Rakefile.
  • Modified the configuration file (.github/workflows/release-on-merge.yml) to adapt to the new build process.

Sherry520 avatar Aug 16 '24 04:08 Sherry520

Everything seems to be working fine. #520 https://github.com/progit/progit2-zh/actions/runs/10416702456/job/28849460914

Sherry520 avatar Aug 16 '24 05:08 Sherry520

现在应该可以build和release成功了,晚上又不小心改动了字体库那边文件的名字,给它恢复并存档了,以后没问题的话,我就不动它了。

Sherry520 avatar Aug 16 '24 15:08 Sherry520