Skip to content

Commit c658909

Browse files
committed
GitHub Actions: install clippy after toolchain
Doing it the other way around means that when we switch to Beta, Clippy won't have been installed on Beta (even though it's already installed on Stable)
1 parent f0acb19 commit c658909

File tree

1 file changed

+5
-6
lines changed

1 file changed

+5
-6
lines changed

.github/workflows/tests.yml

Lines changed: 5 additions & 6 deletions
Original file line numberDiff line numberDiff line change
@@ -126,18 +126,17 @@ jobs:
126126
- name: Checkout code
127127
uses: actions/checkout@v2
128128

129-
# We actually think explicitly installing Clippy isn't necessary,
130-
# as it is currently installed by default.
131-
# But we'll include this step anyway just in case that changes.
132-
- name: Install Clippy
133-
run: rustup component add clippy
134-
135129
- name: Setup toolchain
136130
uses: actions-rs/toolchain@v1
137131
with:
138132
toolchain: ${{ matrix.rust }}
139133
default: true
140134

135+
# Clippy already installed on Stable, but not Beta.
136+
# So, we must install here.
137+
- name: Install Clippy
138+
run: rustup component add clippy
139+
141140
- name: Clippy
142141
env:
143142
CLIPPY: true

0 commit comments

Comments
 (0)