Skip to content

feat: add codspeed status and other status subcommands to get information install#268

Open
GuillaumeLagrange wants to merge 2 commits intomainfrom
cod-2355-add-a-codspeed-setup-status-command
Open

feat: add codspeed status and other status subcommands to get information install#268
GuillaumeLagrange wants to merge 2 commits intomainfrom
cod-2355-add-a-codspeed-setup-status-command

Conversation

@GuillaumeLagrange
Copy link
Contributor

@GuillaumeLagrange GuillaumeLagrange commented Mar 17, 2026

$ codspeed status
  Authentication
    ✓ Logged in
  
  Repository
    ✓ CodSpeedHQ/codspeed (GitHub)
  
  Tools
    ✓ valgrind (valgrind-3.26.0.codspeed)
    ✓ perf (perf version 6.19.7)
    ✓ codspeed-memtrack (memtrack 1.2.2)
  
  System
    codspeed 4.11.1
    nixos 25.11 (x86_64)
    11th Gen Intel(R) Core(TM) i5-11400H @ 2.70GHz (6C / 16GB)

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 17, 2026

Merging this PR will not alter performance

✅ 4 untouched benchmarks


Comparing cod-2355-add-a-codspeed-setup-status-command (9a17932) with main (e1b20ac)

Open in CodSpeed

@GuillaumeLagrange GuillaumeLagrange force-pushed the cod-2355-add-a-codspeed-setup-status-command branch from a41b7b3 to fc2d61e Compare March 18, 2026 10:55
@GuillaumeLagrange GuillaumeLagrange marked this pull request as ready for review March 18, 2026 10:55
@adriencaccia
Copy link
Member

Seen together, let's have codspeed status, codspeed auth status and codspeed setup status.
The codspeed status command should output details about the detected repo if there is one.

@GuillaumeLagrange GuillaumeLagrange force-pushed the cod-2355-add-a-codspeed-setup-status-command branch 2 times, most recently from 4ea949d to 1a13f5c Compare March 18, 2026 22:23
…eed status` commands

- `codspeed auth status`: checks token validity via API, detects
  repository from git remote, and verifies it exists on CodSpeed
- `codspeed setup status`: shows tool installation status for each
  executor via the new `tool_status()` trait method
- `codspeed status`: combines auth, setup, and system info
- Remove noisy warning from `find_repository_root`
@GuillaumeLagrange GuillaumeLagrange force-pushed the cod-2355-add-a-codspeed-setup-status-command branch from 1a13f5c to 9a17932 Compare March 18, 2026 22:31
@GuillaumeLagrange GuillaumeLagrange changed the title feat: add codspeed setup status to get information about tool install feat: add codspeed status and other status subcommands to get information install Mar 18, 2026
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.

2 participants