jwilger/eventcore
1
0
Fork 0
Code Issues 9 Pull requests 2 Projects Releases 62 Activity Actions

docs: Remove performance benchmarks from README #186

Merged
jwilger merged 2 commits from jwilger-patch-1 into main 2025-12-19 13:34:11 -08:00
Conversation 3 Commits 2 Files changed 2 +3 -20
jwilger commented 2025-12-19 13:20:10 -08:00 (Migrated from github.com)
Copy link

Removed performance benchmarks section from README.

These were based on a very early spike of this library by an LLM and their veracity was...questionable. LOL

Removed performance benchmarks section from README. These were based on a very early spike of this library by an LLM and their veracity was...questionable. LOL
claude[bot] (Migrated from github.com) reviewed 2025-12-19 13:20:48 -08:00
claude[bot] (Migrated from github.com) left a comment
Copy link
No description provided.
LGTM. Removing questionable benchmarks is the right call.
copilot-pull-request-reviewer[bot] (Migrated from github.com) reviewed 2025-12-19 13:21:33 -08:00
copilot-pull-request-reviewer[bot] (Migrated from github.com) left a comment
Copy link

Pull request overview

This PR removes a questionable performance benchmarks section from the README that was based on an early LLM-generated spike of the library.

  • Removes 13 lines containing performance metrics (ops/sec, latency, throughput data)

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

## Pull request overview This PR removes a questionable performance benchmarks section from the README that was based on an early LLM-generated spike of the library. - Removes 13 lines containing performance metrics (ops/sec, latency, throughput data) --- 💡 <a href="/jwilger/eventcore/new/main/.github/instructions?filename=*.instructions.md" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Add Copilot custom instructions</a> for smarter, more guided reviews. <a href="https://docs.github.com/en/copilot/customizing-copilot/adding-repository-custom-instructions-for-github-copilot" class="Link--inTextBlock" target="_blank" rel="noopener noreferrer">Learn how to get started</a>.
claude[bot] (Migrated from github.com) approved these changes 2025-12-19 13:33:07 -08:00
claude[bot] (Migrated from github.com) left a comment
Copy link

Clean removal of questionable benchmarks and cleaner CI setup for cargo-audit.

Clean removal of questionable benchmarks and cleaner CI setup for cargo-audit.
Sign in to join this conversation.
Reviewers
No reviewers
claude[bot]
Labels
Clear labels   
adr
Architecture Decision Record

  
automated

   
bug
Something isn't working

  
chore
Maintenance (audits, cleanup)

  
dependencies

   
documentation
Improvements or additions to documentation

  
enhancement
New feature or request

  
epic
Large feature with sub-issues

  
github-actions

   
P1-high

   
P2-medium
Default priority

  
P3-low
Polish, optimization

  
release

   
research
Investigation / spike

  
rust
No labels
adr
automated
bug
chore
dependencies
documentation
enhancement
epic
github-actions
P1-high
P2-medium
P3-low
release
research
rust
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
1 participant
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
jwilger/eventcore!186
No description provided.