Skip to content

Conversation

@CodeMan62
Copy link
Contributor

Description:
replace Option<Vec<Comment>> with Vec<Comment>

BREAKING CHANGE:
N/A

Related issue (if exists):
closes #10531

@CodeMan62 CodeMan62 requested review from a team as code owners July 6, 2025 12:24
@codspeed-hq
Copy link

codspeed-hq bot commented Jul 6, 2025

Merging this PR will not alter performance

✅ 111 untouched benchmarks
⏩ 73 skipped benchmarks1


Comparing CodeMan62:replace-option-vec-comment (15200d6) with main (a54eb0e)

Open in CodSpeed

Footnotes

  1. 73 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

@CodeMan62 CodeMan62 marked this pull request as draft July 6, 2025 12:36
@changeset-bot
Copy link

changeset-bot bot commented Jan 26, 2026

⚠️ No Changeset found

Latest commit: 15200d6

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

1 participant