Skip to content

Commit 7f32c50

Browse files
bowenli86ypatil12
authored andcommitted
perf: install lcov via prebuilt binary (#1123)
**Motivation:** After several rounds of optimization, the entire CI suite is now limited by the `coverage` job, which in turn is constrained by the `install lcov` step. This step currently takes 3.5 minutes to install `lcov` via `apt`. This PR modifies the installation method, switching from `apt` to a prebuilt binary. As a result, the installation time is reduced from 3.5 minutes to just 1 second. **Modifications:** This PR updates the `lcov` installation process to use a prebuilt binary instead of `apt`. **Result:** Reduces the installation time from 3.5 minutes to 1 second.
1 parent 5b98b8c commit 7f32c50

File tree

1 file changed

+4
-3
lines changed

1 file changed

+4
-3
lines changed

.github/workflows/foundry.yml

Lines changed: 4 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -181,10 +181,11 @@ jobs:
181181
version: stable
182182

183183
# Install LCOV for coverage report generation.
184-
- name: Install LCOV
184+
- name: Install LCOV (Prebuilt)
185185
run: |
186-
sudo apt-get install -y lcov
187-
id: lcov
186+
curl -L https://github.com/linux-test-project/lcov/releases/download/v1.16/lcov-1.16.tar.gz | tar xz
187+
sudo cp lcov-1.16/bin/* /usr/local/bin/
188+
sudo cp -r lcov-1.16/man/* /usr/share/man/
188189
189190
# Build the project and display contract sizes.
190191
- name: "Forge Build"

0 commit comments

Comments
 (0)