Skip to content

feat: add codspeed walltime benchmark#59

Closed
stormslowly wants to merge 5 commits into
mainfrom
perf/codspeed_walltime
Closed

feat: add codspeed walltime benchmark#59
stormslowly wants to merge 5 commits into
mainfrom
perf/codspeed_walltime

Conversation

@stormslowly
Copy link
Copy Markdown
Collaborator

No description provided.

@codspeed-hq
Copy link
Copy Markdown

codspeed-hq Bot commented Mar 31, 2025

CodSpeed Performance Report

Merging #59 will not alter performance

Comparing perf/codspeed_walltime (4779e17) with main (cc908e8)

Summary

✅ 2 untouched benchmarks

@SoonIter
Copy link
Copy Markdown
Member

SoonIter commented Apr 1, 2025

what is ferf?

@stormslowly stormslowly changed the title ferf: add codspeed walltime benchmark fert: add codspeed walltime benchmark Apr 1, 2025
@stormslowly stormslowly changed the title fert: add codspeed walltime benchmark feat: add codspeed walltime benchmark Apr 1, 2025
@stormslowly stormslowly marked this pull request as draft April 1, 2025 09:06
Comment on lines +59 to +92
name: Wall-time
runs-on: codspeed-macro
steps:
- name: Checkout Branch
uses: taiki-e/checkout-action@v1

- uses: pnpm/action-setup@v4

- uses: actions/setup-node@v4
with:
node-version-file: .node-version
cache: pnpm

- name: Setup benchmark data
run: cd benches && pnpm install --ignore-workspace

- uses: Boshen/setup-rust@main
with:
cache-key: benchmark
save-cache: ${{ github.ref_name == 'main' }}
tools: cargo-codspeed

- uses: ./.github/actions/pnpm
- name: Build Benchmark
env:
RUSTFLAGS: "-C debuginfo=1 -C strip=none -g"
run: cargo codspeed build

- name: Run benchmark
uses: CodSpeedHQ/action@v3
timeout-minutes: 30
with:
run: cargo codspeed run
token: ${{ secrets.CODSPEED_TOKEN }}

Check warning

Code scanning / CodeQL

Workflow does not contain permissions

Actions Job or Workflow does not set permissions
@stormslowly
Copy link
Copy Markdown
Collaborator Author

walltime feature need orgnization's admin permission so pending this for latest codspeed update

@adriencaccia
Copy link
Copy Markdown

Hey @stormslowly, as stated on Crisp, the permission has been removed from the CodSpeed GitHub app. We now rely on organization self-hosted runners instead of repository ones.
You should be able to install the app now, I hope it goes smoothly!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants