Skip to content

Commit 1e8513a

Browse files
committed
AI review changes
1 parent a27ea8b commit 1e8513a

File tree

2 files changed

+7
-5
lines changed

2 files changed

+7
-5
lines changed

README.md

Lines changed: 6 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -158,12 +158,13 @@ To provide feedback (requesting a feature or reporting a bug) please post to [is
158158

159159
## Required tools installed
160160

161-
As of v2.16.0, this action now uses
161+
As of v2.16.0, this action uses
162162

163163
- [nushell] for cross-platform compatible scripting
164164
- [uv] for driving a Python virtual environment
165165

166-
This action installs [nushell] and [uv] automatically, but only [nushell] is added to the PATH environment variable;
166+
This action installs [nushell] and [uv] automatically.
167+
Only [nushell] is added to the PATH environment variable.
167168
[uv], and any standalone Python distribution it downloads, are not added to the PATH environment variable.
168169

169170
### On Linux runners
@@ -172,7 +173,7 @@ We only support Linux runners using a Debian-based Linux OS (like Ubuntu and man
172173
This is because we first try to use the `apt` package manager to install clang tools.
173174

174175
Linux workflows that use a specific [`container`][gh-container-syntax] should ensure that
175-
the following is installed:
176+
the following are installed:
176177

177178
- GLIBC (v2.32 or later)
178179
- `wget` or `curl`
@@ -181,12 +182,13 @@ the following is installed:
181182
- `gnupg` (required by LLVM-provided install script)
182183

183184
```shell
185+
apt-get update
184186
apt-get install -y libc6 wget lsb-release software-properties-common gnupg
185187
```
186188

187189
Otherwise, [nushell] and/or the LLVM-provided bash script will fail to run.
188190

189-
### On MacOS runners
191+
### On macOS runners
190192

191193
The specified `version` of `clang-format` and `clang-tidy` is installed via Homebrew.
192194
Failing that, we attempt to use static binaries that we built ourselves;

action.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -428,7 +428,7 @@ runs:
428428
}
429429
430430
let local_bin = '~/.local/bin' | path expand
431-
let tmp_path = if (
431+
if (
432432
('${{ runner.os }}' == 'Linux')
433433
and ($local_bin | path exists)
434434
and (not ($env.PATH | any {$in == $local_bin}))

0 commit comments

Comments
 (0)