Closed YugenTakadaka closed 1 year ago
install.ps1 は layerdivider ディレクトリに venv ディレクトリを作成することを意図して作られたのだと思います。 しかしながら、PowerShell から 直接 install.ps1 を実行する場合、カレントディレクトリの場所に venv ディレクトリが作成されます。 これは意図とは異なる動作を引き起こす可能性があります。 例えば、PowerShell で
cd C:\ git clone https://github.com/mattyamonaca/layerdivider.git .\layerdivider\install.ps1
とすると、ディレクトリは C:\ ├ layerdivider\ └ venv\ となってしまいます。
install.ps1 の先頭行にSet-Location $PSScriptRootを追加する。
Set-Location $PSScriptRoot
install.ps1 を実行する方法にかかわらず同じディレクトリ構成になるので、利用者にとって分かりやすいと思います。
もともとの挙動を利用している人がいるかもしれません。
問題
install.ps1 は layerdivider ディレクトリに venv ディレクトリを作成することを意図して作られたのだと思います。 しかしながら、PowerShell から 直接 install.ps1 を実行する場合、カレントディレクトリの場所に venv ディレクトリが作成されます。 これは意図とは異なる動作を引き起こす可能性があります。 例えば、PowerShell で
とすると、ディレクトリは C:\ ├ layerdivider\ └ venv\ となってしまいます。
提案
install.ps1 の先頭行に
Set-Location $PSScriptRoot
を追加する。メリット
install.ps1 を実行する方法にかかわらず同じディレクトリ構成になるので、利用者にとって分かりやすいと思います。
デメリット
もともとの挙動を利用している人がいるかもしれません。