Skip to content

Conversation

@RiskyMH
Copy link
Member

@RiskyMH RiskyMH commented Aug 14, 2025

What does this PR do?

followup #21833

How did you verify your code works?

- Replace std.ArrayList with std.fifo.LinearFifo for queue in buildProductionPackageSet
- Replace multiple if statements with ComptimeStringMap in AuditLevel.fromString

🤖 Generated with Claude Code

Co-Authored-By: Claude <[email protected]>
@robobun
Copy link
Collaborator

robobun commented Aug 14, 2025

Updated 6:42 PM PT - Aug 14th, 2025

@RiskyMH, your commit 676efaa has 3 failures in Build #22996:


🧪   To try this PR locally:

bunx bun-pr 21873

That installs a local version of the PR into your bun-21873 executable, so you can run:

bun-21873 --bun

RiskyMH and others added 4 commits August 14, 2025 16:45
- Use cleaner bounds checking with 'and' operator
- Remove redundant DynamicBitSet tracking
- Keep the LinearFifo improvement from PR feedback

🤖 Generated with Claude Code

Co-Authored-By: Claude <[email protected]>
@Jarred-Sumner Jarred-Sumner merged commit 447f844 into main Aug 15, 2025
59 of 60 checks passed
@Jarred-Sumner Jarred-Sumner deleted the riskymh/bun-audit-filtering branch August 15, 2025 02:51
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