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

ci: ヘルプ生成GHAの改善点 #1011

Open
tsuyoshicho opened this issue Jan 22, 2022 · 2 comments
Open

ci: ヘルプ生成GHAの改善点 #1011

tsuyoshicho opened this issue Jan 22, 2022 · 2 comments
Assignees

Comments

@tsuyoshicho
Copy link
Contributor

課題

いくつかの点で改善が必要そうというメモ & 良いやり方にできそうなら対応はしたい

GHA自体

  • fork先でもエラーしないで動いてしまう(pushはできていない) : 本来はエラーするべきのはず

これについては下参照

EndBug/add-and-commit@v7 の利用関係

  • pathspec_error_handling の指定がデフォルトの ignore のため、 git add 操作のNGは検知できない状態
  • 現在排他関係でworkflowとしてシリアライズしている、それは維持するとしても pull: 'NO-PULL' は pull: '--rebase' にするのがよいかもしれない
  • output パラメータの pushed を見て push 失敗でfailさせたほうがよさそう(ただしヘルプに変更がなかった時はpushがなく、CIでエラーする状況になるかもなので注意)

参考

- name: Commit updated master branch
uses: EndBug/add-and-commit@v7
with:
cwd: './target'
default_author: github_actions
message: |-
Generated by GitHub workflow ${{ github.run_id }}
https://github.com/${{ github.repository }}/actions/runs/${{ github.run_id }}
branch: master
pull: 'NO-PULL'

@tsuyoshicho tsuyoshicho self-assigned this Jan 22, 2022
@k-takata
Copy link
Member

EndBug/add-and-commit を使わず普通にgitコマンドを使えばいいのではという気もします。

@tsuyoshicho
Copy link
Contributor Author

そうしたほうがいいかもですね...

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

No branches or pull requests

2 participants