Skip to content

feat(CI): Set up CodSpeed - #239

Merged
emyller merged 2 commits into
mainfrom
feat/codspeed
Oct 29, 2026
Merged

emyller merged 2 commits into
mainfrom
feat/codspeed

Conversation

@emyller

@emyller emyller commented Oct 29, 2026

Copy link
Copy Markdown
Contributor

Closes SDK-15.

Set up CodSpeed in GitHub Actions.

🤖 Generated with Claude Code

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@emyller
emyller requested a review from a team as a code owner October 29, 2026 20:18
@emyller
emyller requested review from khvn26 and removed request for a team October 29, 2026 20:18
@linear-code

linear-code Bot commented Oct 29, 2026

Copy link
Copy Markdown

SDK-15

@coderabbitai

coderabbitai Bot commented Oct 29, 2026 •

Copy link
Copy Markdown
Contributor

Review in Change Stack →

Warning

Review limit reached

You've used all free OSS reviews for now. Wait for the free limit to reset to keep reviewing this public repository.

Next included review available in 24 minutes.

Check out review usage here.

View limit details

Limit details: You’ve used the included review currently available.

Learn how review limits work.

Review configuration:

Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: cb483a18-d554-474c-b91c-8a8004b129d8
Commits

Reviewing files that changed from the base of the PR and between 0ef48ab and 4ce1a85.

Files selected for processing: 1
  • .github/workflows/codspeed.yml
Walkthrough

Adds Go benchmarks for environment flag lookup, identity flag lookup, and identity segment lookup across environment documents. Adds a Make target that initialises the benchmark data submodule and runs the benchmarks. Adds a CodSpeed workflow to run the target on pushes to main, pull requests, and manual dispatch.

Priority: Low

Estimated code review effort: 3 (Moderate) | ~20 minutes

Merge Risk: Low · previous review 0ef48

Remote submodule updates fail, and Go upgrades can affect benchmark comparisons. The normal benchmark workflow remains usable, so these are bounded follow-ups.

- [ ] Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@codspeed

codspeed Bot commented Oct 29, 2026 •

Copy link
Copy Markdown

Congrats! CodSpeed is installed 🎉

🆕 15 new benchmarks were detected.

You will start to see performance impacts in the reports once the benchmarks are run from your default branch.

Detected benchmarks


Open in CodSpeed

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Actionable comments posted: 2


Review info
Run configuration
  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: d472a823-9044-4bbb-b1f6-ddf45d5bf0ee
Commits

Reviewing files that changed from the base of the PR and between cd7e433 and 0ef48ab.

Files selected for processing: 5
  • .github/workflows/codspeed.yml
  • .gitmodules
  • Makefile
  • benchmarks/sdk_benchmark_test.go
  • flagengine/sdk-benchmarks

Included review availability: This review used your included allowance. Your plan provides up to 1 included review per hour; 0 remain after this review.

Comment thread .github/workflows/codspeed.yml Resolved
Comment thread .gitmodules Resolved
Co-authored-by: coderabbitai[bot] <136622811+coderabbitai[bot]@users.noreply.github.com>
@emyller
emyller merged commit adc44cb into main Oct 29, 2026
7 checks passed
@emyller
emyller deleted the feat/codspeed branch October 29, 2026 21:11
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