ファイル名にスペースが入ってると prettier-format.yml がコケる

[sozysozbot]

https://github.com/jurliyuuri/cerke_online_alpha/runs/4282872483?check_suite_focus=true で

fatal: pathspec '"media_drafts/tateyoko' did not match any files

と書いてある。これの正体は media_drafts/tateyoko - コピー.png。

[sozysozbot]

@azarashi2931 曰く、「prettier をローカルで走らせたら普通に動いた。悪いのは prettier じゃなくて GitHub Action 側で、only_changed: True で変更されたファイル一覧を取ってきているのだろうが、その際にスペースを考慮していなくてコケている」

https://github.com/jurliyuuri/cerke_online_alpha/blob/master/.github/workflows/prettier-format.yml#L29

[sozysozbot]

@azarashi2931 曰く、「さっきのは誤りかもしれない」

[azrsh]

あやしい https://github.com/creyD/prettier_action/blob/0b655e298cc5c8477c9df9e60dfb6cd5d79f3055/entrypoint.sh#L96-L99

[sozysozbot]

@azarashi2931 曰く、「さっきのは実は合ってたかもしれない」

[azrsh]

エラー自体はGitが吐いているものっぽい

[sksat]

ダルいわね〜

[sksat]

え，この問題はダルいとして，このファイルはいるんですか

[sozysozbot]

いいところに気がつきましたね。実はわりと用済みです。media_drafts は「後で使い回せる可能性があるが、現時点ではデプロイ成果物に含めなくていい画像」を入れておく場所であり、この画像はマジで使い回す未来が見えないので削ってよい

[sozysozbot]

削った ef02faf

[sksat]

これでよいのでは sksat/prettier_action@b90e277

[sksat]

PR，反応無し！(まあしゃーない)

[sozysozbot]

そういうこともあるよね

[sksat]

マージされてv4.1から入ったみたいです
creyD/prettier_action#73

[sozysozbot]

やった～～

[sozysozbot]

まだ解決はしてないので reopen

[sksat]

なんでか知らんけどv.4.0すらpre releaseっぽい．なぜ？　https://github.com/creyD/prettier_action/releases

[sksat]

まあv4.0でなんの問題も無いと思うので更新してよさそう．Renovateくんはやってきてくれないけれど...(Releaseじゃなくてtagで見てもらうなどの手はあるが，別にリリース打たないというわけではないと思うのでそれにするのは違う)

[sksat]

tag見てやるか〜？と思いつつ，preとはいえRelease出てんだからその文言は出てほしい．そこで気付き: RenovateくんはPre ReleaseでもPR送ってくれるオプション無いのか？

[sksat]

というか5日前にdraft releaseをskipするやつがマージされとるな．いけるのでは？
renovatebot/renovate#12856

[sksat]

prerelease用の変数はおるねんな: https://github.com/renovatebot/renovate/blob/dd7bb46723ec16c0e4cbcd0dbf56bc8ab692d8c2/lib/datasource/github-releases/types.ts#L5

[sksat]

isStableとかいうやつになってるので，stableでなくてもいいみたいなオプションがあればよさそうだな
https://github.com/renovatebot/renovate/blob/dd7bb46723ec16c0e4cbcd0dbf56bc8ab692d8c2/lib/datasource/github-releases/index.ts#L64

[sksat]

これログ見たらdatasource github-tagsになってるじゃん．なぜ？
https://app.renovatebot.com/dashboard#github/jurliyuuri/cerke_online_alpha/519664329

[sksat]

そういやコイツもバージョニングがカスなので4.0や4.1にはなっても4.1.1にはならんだろうな．まず4.0にぐらいなってほしいんですけどね．

[sksat]

一旦手動で4.0にして様子を見るなどはあるな

[sksat]

手動で4.0にしてre runしたがPRこねえ！カス！