download_test.ymlのmatrix.expected_versionは不要なのではないか

[qryxip]

内容

download_test.ymlにactionlintをかけたらmatrix.expected_versionがどこにも定義されていないことに気づきました。

#249 の経緯があまりよくわかっていないのですが、既に不要な部分ではないかと思います。

❯ actionlint .github/workflows/download_test.yml
.github/workflows/download_test.yml:238:43: property "expected_version" is not defined in object type {check_items: string; check_not_exists_items: string; download_command: string; download_dir: string; name: string; os: string} [expression]
    |
238 |       EXPECTED_VOICEVOX_CORE_VERSION: ${{ matrix.expected_version || 'latest' }}
    |                                           ^~~~~~~~~~~~~~~~~~~~~~~
.github/workflows/download_test.yml:246:9: shellcheck reported issue in this script: SC2086:info:1:231: Double quote to prevent globbing and word splitting [shellcheck]
    |
246 |         run: |
    |         ^~~~
.github/workflows/download_test.yml:255:9: shellcheck reported issue in this script: SC2207:warning:1:8: Prefer mapfile or read -a to split command output (or quote to avoid splitting) [shellcheck]
    |
255 |         run: |
    |         ^~~~
.github/workflows/download_test.yml:255:9: shellcheck reported issue in this script: SC2086:info:5:37: Double quote to prevent globbing and word splitting [shellcheck]
    |
255 |         run: |
    |         ^~~~
.github/workflows/download_test.yml:264:9: shellcheck reported issue in this script: SC2207:warning:1:8: Prefer mapfile or read -a to split command output (or quote to avoid splitting) [shellcheck]
    |
264 |         run: |
    |         ^~~~
.github/workflows/download_test.yml:264:9: shellcheck reported issue in this script: SC2086:info:5:39: Double quote to prevent globbing and word splitting [shellcheck]
    |
264 |         run: |
    |         ^~~~

voicevox_core/.github/workflows/download_test.yml

Lines 237 to 252 in af9cf97

	env:
	EXPECTED_VOICEVOX_CORE_VERSION: ${{ matrix.expected_version || 'latest' }}
	steps:
	- uses: actions/checkout@v2
	- name: Execute download command
	run: ${{ matrix.download_command }}
	- name: Get latest version
	if: ${{ env.EXPECTED_VOICEVOX_CORE_VERSION == 'latest' }}
	shell: bash
	run: |
	echo "EXPECTED_VOICEVOX_CORE_VERSION=$(curl -sSfI "https://github.com/VOICEVOX/voicevox_core/releases/latest"| grep "location:" | sed -e "s%location: https://github.com/VOICEVOX/voicevox_core/releases/tag/%%" | sed 's/\r//g')" >> $GITHUB_ENV
	- name: Check downloaded version
	shell: bash
	run: |
	[ -e "${{ matrix.download_dir }}/VERSION" ]
	[ "$(cat "${{ matrix.download_dir }}/VERSION")" = "${{ env.EXPECTED_VOICEVOX_CORE_VERSION }}" ]

Pros 良くなる点

Cons 悪くなる点

実現方法

OSの種類/ディストリ/バージョン

その他

[Hiroshiba]

おそらくダウンロードバージョンを指定可能にするというモチベーションがあったのかなと思いました！
となると、どちらかというと指定可能になっていない点が課題かも･･･？

[qwerty2501]

ダウンロードスクリプトがリリースされた後の過去バージョン指定してダウンロードできることを確認するモチベがあったため書いたのですが、まだその過去バージョンがないためケースとしては追加しなかった記憶があります。
lint通すのが目的ならとりあえず消しても良いかも？

[Hiroshiba]

なるほどです、となるとコメントアウトして目的を書けば良さそうですね･･･！