build: tests and more checks #5268

2025-08-29T15:37:36Z

delliott commented

2025-08-29 15:37:36 +00:00

add pytest
add tests for all exposed lib functions
Add a whole bunch more ruff checks and apply them

Benchmarking:

Before:

❯ hyperfine --runs 3 "uv run build --full"
Benchmark 1: uv run build --full
  Time (mean ± σ):     287.463 s ± 15.161 s    [User: 1573.732 s, System: 72.548 s]
  Range (min … max):   272.953 s … 303.201 s    3 runs

After

❯ hyperfine --runs 3 "uv run build --full"
Benchmark 1: uv run build --full
  Time (mean ± σ):     280.184 s ±  9.605 s    [User: 1557.788 s, System: 72.693 s]
  Range (min … max):   269.878 s … 288.886 s    3 runs

So we gain a few seconds, but that is all.

The main benefit is that we have better testing and more linting.

- add pytest - add tests for all exposed lib functions - Add a whole bunch more ruff checks and apply them Benchmarking: Before: ``` ❯ hyperfine --runs 3 "uv run build --full" Benchmark 1: uv run build --full Time (mean ± σ): 287.463 s ± 15.161 s [User: 1573.732 s, System: 72.548 s] Range (min … max): 272.953 s … 303.201 s 3 runs ``` After ``` ❯ hyperfine --runs 3 "uv run build --full" Benchmark 1: uv run build --full Time (mean ± σ): 280.184 s ± 9.605 s [User: 1557.788 s, System: 72.693 s] Range (min … max): 269.878 s … 288.886 s 3 runs ``` So we gain a few seconds, but that is all. The main benefit is that we have better testing and more linting.

delliott marked the pull request as work in progress 2025-08-29 15:37:42 +00:00

delliott force-pushed feat/pytest from 74f7bd822a to 37dd03b891

2025-08-29 15:47:29 +00:00

Compare

delliott requested review from tobiasd 2025-08-29 15:47:45 +00:00

delliott marked the pull request as ready for review 2025-08-29 15:47:46 +00:00

delliott added 1 commit 2025-08-29 16:33:15 +00:00

misc: run pytest in pre-commit

continuous-integration/drone/pr Build is passing

Details

continuous-integration/drone/push Build is passing

Details

d27ed4b44d

delliott added 1 commit 2025-08-29 16:43:47 +00:00

ci: force resinatll of correct package

continuous-integration/drone/pr Build is passing

Details

continuous-integration/drone/push Build is passing

Details

c1e95dd555

delliott merged commit f03b3c2c4c into master

2025-08-29 16:46:20 +00:00

delliott referenced this issue from a commit

2025-08-29 16:46:21 +00:00

build: tests and more checks (#5268)

Sign in to join this conversation.

1 Participants

Notifications

Due Date

No due date set.

Dependencies

No dependencies set.

Reference: FSFE/fsfe-website#5268