-
Notifications
You must be signed in to change notification settings - Fork 65
fix: report workflow #1148
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: report workflow #1148
Conversation
すんません。#1134 を何も考えずにmergeしちゃいました 🙇♂️ |
あ、とすると自分も間違えてますね...なおします。 |
tools/script/get_doc_info.sh
Outdated
@@ -7,7 +7,7 @@ | |||
|
|||
# 自分の環境にあわせて変更する必要がある |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
コメントも変えた方がいいかもですね。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
3行目のコメントとここのパスが違っていたから「修正しよう」と思い立ったのだと思いますんで、よしなに変更ください。
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
たしかに、手直ししておきます。
Thanks for your follow up:+1: |
close #1147
差分レポートを修正
いままで、デフォルトで clone されている内容から差分を生成していた。おそらくデフォルトの clone がされなくなったために失敗するようになった。
#1134 での変更で対象が上手く選択できなくなった。
workflow で clone している work 以下を正しく利用するように変更
step: Checkout vimdoc-ja-working にて
https://github.com/vim-jp/vimdoc-ja-working/actions/runs/3165169613
なのに
step: Build report の get_doc_info.sh
での定義が
これにより
という失敗