Skip to content

Commit 78763d9

Browse files
authored
add -y to any apt-get install commands (#149)
I checked the LLVM install script, and it already uses `apt-get install -y`. So this should resolve #148
1 parent 14dc26d commit 78763d9

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

action.yml

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -91,12 +91,12 @@ runs:
9191
if [[ "${{runner.os}}" == "Linux" ]]; then
9292
sudo apt-get update
9393
# First try installing from default Ubuntu repositories before trying LLVM script
94-
if ! sudo apt-get install clang-format-${{ inputs.version }} clang-tidy-${{ inputs.version }}; then
94+
if ! sudo apt-get install -y clang-format-${{ inputs.version }} clang-tidy-${{ inputs.version }}; then
9595
# This LLVM script will add the relevant LLVM PPA: https://apt.llvm.org/
9696
wget https://apt.llvm.org/llvm.sh -O $GITHUB_ACTION_PATH/llvm_install.sh
9797
chmod +x $GITHUB_ACTION_PATH/llvm_install.sh
9898
if sudo $GITHUB_ACTION_PATH/llvm_install.sh ${{ inputs.version }}; then
99-
sudo apt-get install clang-format-${{ inputs.version }} clang-tidy-${{ inputs.version }}
99+
sudo apt-get install -y clang-format-${{ inputs.version }} clang-tidy-${{ inputs.version }}
100100
fi
101101
fi
102102
fi

0 commit comments

Comments
 (0)