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

github actionsの更新、およびtextlintを再帰的にmdを確認するようにした。 #45

Closed ghost closed 3 years ago

equal-l2 commented 3 years ago

Actionsのログ見た感じ、/usr/localへのコピーも権限不足でこけてるみたいなので、その部分にsudoつけてもらえますか?

https://github.com/su-its/ams-backend-nodejs/runs/2078422664?check_suite_focus=true#step:5:3181

equal-l2 commented 3 years ago

しょうがないことではあるんですが、aptすごい時間かかりますね……

ghost commented 3 years ago

@equal-l2 ちゃんと動いたよ~

equal-l2 commented 3 years ago

現状では具体的な指摘はCIの範囲外ってことでいいんですよね? (CI通らなかったら各自手元でredpen,textlintを実行して指摘を修正する……というワークフロー?)

ghost commented 3 years ago

@equal-l2 いや、PRのauthorだったらmerge出来るっぽいけど、mergeしちゃって大丈夫ですかね...

というか、Docker回したら一応、localでRedpen回るようにはしたのですが

equal-l2 commented 3 years ago

了解です。 ありがとうございました。

ghost commented 3 years ago

@equal-l2 あ、ごめんなさい、頓珍漢な事言ってました。

そうですね、ワークフローではそのつもりだったんですが、 何故かというと、APIはあらかた完成していてdocsを書くことを主軸にしてもいいかな、と思ってました。ごめんなさい。

equal-l2 commented 3 years ago

CIの結果をGitHubで表示するのは結構面倒なので、そういうワークフローで全然問題ないです。

ghost commented 3 years ago

@equal-l2 了解です。