Skip to content
This repository has been archived by the owner on Jun 1, 2023. It is now read-only.

GitHub Actionsのパスを統一する #6921

Open
Murayu0225 opened this issue Nov 10, 2021 · 2 comments
Open

GitHub Actionsのパスを統一する #6921

Murayu0225 opened this issue Nov 10, 2021 · 2 comments
Labels
improvement 改善や新機能の要望

Comments

@Murayu0225
Copy link
Contributor

Murayu0225 commented Nov 10, 2021

改善詳細 / Details of Improvement

  • この記事によるとGitHub Actionのパスは、GITHUB_WORKSPACEと書くことによって省略できるようです。

このリポジトリを確認したところ、所々にGITHUB_WORKSPACEを用いて書いてあったりしましたが、書いていないところも多々ありました。提案となるのですが、パスには全てGITHUB_WORKSPACEを用いて書くのはどうでしょうか。

↓一応、workflow類をGITHUB_WORKSPACEにして作ってみました。

https://github.com/Murayu0225/covid19-1/tree/fix-action-path

(Actionsが正常に通っているかは確認中です。)

.envに関してはパスを一貫してGITHUB_WORKSPACEにするために${{ github.workspace }}/.envとしています。.envはそのままの方がよければ修正します。

https://github.com/Murayu0225/covid19-1/blob/fix-action-path/.github/workflows/deploy.yml#L61-L61

[追記]
思っていたよりもビルドが通らないものが多かったのでPR作成まで時間がかかりそうです。

@Murayu0225 Murayu0225 added the improvement 改善や新機能の要望 label Nov 10, 2021
@kaizumaki
Copy link
Collaborator

@Murayu0225 ご提案ありがとうございます!おっしゃるとおり、書き方は統一しておいたほうがいいですね。
ある程度できたらDraftでPRを出してみてもらえると助かります 🙏

@Murayu0225
Copy link
Contributor Author

わかりました。#6922 にて提出しました。

よろしくお願いします。

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
improvement 改善や新機能の要望
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants