su-its / ams-backend

:briefcase: (This repository is no longer maintained) The backend server of our Access-management-system.
MIT License
0 stars 0 forks source link

[WIP]Docs_CI_Test #60

Closed ghost closed 3 years ago

ghost commented 3 years ago

github actions が仕様変更が多すぎて、リバースエンジニアリングして変数名を考えているところです。

equal-l2 commented 3 years ago

INPUT_***で書いてる部分、各actionsの説明書に書いてある小文字の変数名で書いても勝手に変換されるので、そちらで書いてもよさそうです。 https://docs.github.com/en/actions/reference/workflow-syntax-for-github-actions#jobsjob_idstepswith

ghost commented 3 years ago

@equal-l2 github acrions の設定のパラメータがverごとに違うみたいで、github_tokenはGITHUB_TOKENに

最近変更らしいのですが、どれが大文字・小文字関係ないのかはわかってないので、もらったURLちゃんと読んでおきます

equal-l2 commented 3 years ago

github_tokenですが、やはり省略できそうです。 自分の使った action-eslint でも github_tokenrequired と書いてありましたが、デフォルト値が指定されているので省略できました。

念のためこのPRで使われている全てのactionの action.yml を見てきましたが、どれでも github_token にはデフォルト値が指定されているので、省略できるものと思われます。 (デフォルト値があるのに required というのがそもそも変な話ですが……)


以下、action.ymlの該当部分です。 https://github.com/tsuyoshicho/action-redpen/blob/7ecc7d73d5d2fcb9d3be59486d338f0c19689ecb/action.yml#L5-L8

https://github.com/tsuyoshicho/action-textlint/blob/4c4a295e176ba3e475a594ba3e568a8bc231ecce/action.yml#L5-L8

https://github.com/reviewdog/action-alex/blob/91ea364b11599c1e6b96bab6a41ab2990432e1a5/action.yml#L5-L7

ghost commented 3 years ago

@equal-l2 了解です。actionsはreviewdogの仕様が難しいっぽいですね

ghost commented 3 years ago

これはヤバいので辞めておきます。