Skip to content

Use OIDC instead of codspeed token#21719

Merged
MichaReiser merged 3 commits intomainfrom
micha/token-free-codspeed
Dec 1, 2025
Merged

Use OIDC instead of codspeed token#21719
MichaReiser merged 3 commits intomainfrom
micha/token-free-codspeed

Conversation

@MichaReiser
Copy link
Member

@MichaReiser MichaReiser commented Dec 1, 2025

Don't use a persistent token, instead use OIDC to generate a token on the fly, as recommended by codspeed.

I also updated the mode from instrumentation to simulation because codspeed logged a deprecation warning

@MichaReiser MichaReiser added the ci Related to internal CI tooling label Dec 1, 2025
@MichaReiser MichaReiser requested a review from woodruffw December 1, 2025 07:28
@astral-sh-bot
Copy link

astral-sh-bot bot commented Dec 1, 2025

ruff-ecosystem results

Linter (stable)

✅ ecosystem check detected no linter changes.

Linter (preview)

✅ ecosystem check detected no linter changes.

Formatter (stable)

✅ ecosystem check detected no format changes.

Formatter (preview)

✅ ecosystem check detected no format changes.

@MichaReiser MichaReiser marked this pull request as ready for review December 1, 2025 07:28
Copy link
Member

@woodruffw woodruffw left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice! Just flagging for diligence that CODSPEED_TOKEN should be removed from the actions secrets once this is merged too 🙂

@MichaReiser MichaReiser merged commit b4f618e into main Dec 1, 2025
38 checks passed
@MichaReiser MichaReiser deleted the micha/token-free-codspeed branch December 1, 2025 16:51
dcreager added a commit that referenced this pull request Dec 2, 2025
* origin/main: (67 commits)
  Move `Token`, `TokenKind` and `Tokens` to `ruff-python-ast` (#21760)
  [ty] Don't confuse multiple occurrences of `typing.Self` when binding bound methods (#21754)
  Use our org-wide Renovate preset (#21759)
  Delete `my-script.py` (#21751)
  [ty] Move `all_members`, and related types/routines, out of `ide_support.rs` (#21695)
  [ty] Fix find-references for import aliases (#21736)
  [ty] add tests for workspaces (#21741)
  [ty] Stop testing the (brittle) constraint set display implementation (#21743)
  [ty] Use generator over list comprehension to avoid cast (#21748)
  [ty] Add a diagnostic for prohibited `NamedTuple` attribute overrides (#21717)
  [ty] Fix subtyping with `type[T]` and unions (#21740)
  Use `npm ci --ignore-scripts` everywhere (#21742)
  [`flake8-simplify`] Fix truthiness assumption for non-iterable arguments in tuple/list/set calls (`SIM222`, `SIM223`) (#21479)
  [`flake8-use-pathlib`] Mark fixes unsafe for return type changes (`PTH104`, `PTH105`, `PTH109`, `PTH115`) (#21440)
  [ty] Fix auto-import code action to handle pre-existing import
  Enable PEP 740 attestations when publishing to PyPI (#21735)
  [ty] Fix find references for type defined in stub (#21732)
  Use OIDC instead of codspeed token (#21719)
  [ty] Exclude `typing_extensions` from completions unless it's really available
  [ty] Fix false positives for `class F(Generic[*Ts]): ...` (#21723)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Related to internal CI tooling

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants