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

テストが通るように修正 #60

Merged
merged 33 commits into from
Nov 19, 2024
Merged

テストが通るように修正 #60

merged 33 commits into from
Nov 19, 2024

Conversation

MasayaMORIMOTO
Copy link
Contributor

@MasayaMORIMOTO MasayaMORIMOTO commented Nov 1, 2024

関連 Issue

#49

背景

プルリクを作成した際に走る、PHP Unit Test が通るように、関連ファイルを修正

@MasayaMORIMOTO MasayaMORIMOTO changed the title テストが通るように修正 【確認待ち】テストが通るように修正 Nov 1, 2024
@MasayaMORIMOTO MasayaMORIMOTO self-assigned this Nov 1, 2024
@drill-lancer
Copy link
Member

@MasayaMORIMOTO

sudo curl -sL https://github.com/docker/compose/releases/download/v2.14.2/docker-compose-linux-x86_64 -o /usr/local/bin/docker-compose
        sudo chmod 755 /usr/local/bin/docker-compose

この2行はいらないような気がしますが・・・
他のリポジトリでも同様の記載はありません。

@MasayaMORIMOTO
Copy link
Contributor Author

@drill-lancer
sudo curl -sL https://github.com/docker/compose/releases/download/v2.14.2/docker-compose-linux-x86_64 -o /usr/local/bin/docker-compose sudo chmod 755 /usr/local/bin/docker-compose
おっしゃる通り、こちらの2行は、他のリポジトリでも同様の記載はありませんが、
この2行がないと、下記画像のように、Unit Test がこけるのです。

スクリーンショット 2024-11-06 16 00 56

@drill-lancer
Copy link
Member

@wordpress/env のバージョンが古いのが原因のようです。(最新版は 10.11.0)

@MasayaMORIMOTO
Copy link
Contributor Author

@drill-lancer
@wordpress/env のバージョンのバージョンを上げて対応しました。
お手数ですが、再度ご確認をお願いいたします。

@drill-lancer
Copy link
Member

drill-lancer commented Nov 13, 2024

@MasayaMORIMOTO
問題ないと思いますが一部のコマンドが不必要に長かった気がしたので修正しておきました。


どなたか2人目確認お願いします。

@drill-lancer drill-lancer changed the title 【確認待ち】テストが通るように修正 【2人目確認待ち】テストが通るように修正 Nov 13, 2024
@kurudrive kurudrive changed the title 【2人目確認待ち】テストが通るように修正 テストが通るように修正 Nov 19, 2024
@kurudrive kurudrive merged commit 031022a into master Nov 19, 2024
3 checks passed
@kurudrive kurudrive deleted the fix/unit-test branch November 19, 2024 07:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants