Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

docs: Reorganize docs for xxx_with for better reading #5517

Merged
merged 1 commit into from
Jan 7, 2025

Conversation

Xuanwo
Copy link
Member

@Xuanwo Xuanwo commented Jan 7, 2025

Which issue does this PR close?

Follow up of #5516

image

Rationale for this change

What changes are included in this PR?

Are there any user-facing changes?

Signed-off-by: Xuanwo <github@xuanwo.io>
@Xuanwo Xuanwo merged commit 4fe235a into main Jan 7, 2025
99 checks passed
@Xuanwo Xuanwo deleted the polish-docs-again branch January 7, 2025 11:30
Copy link

codspeed-hq bot commented Jan 7, 2025

CodSpeed Performance Report

Merging #5517 will improve performances by 18.88%

Comparing polish-docs-again (03a145b) with main (b84172e)

Summary

⚡ 4 improvements
✅ 69 untouched benchmarks

Benchmarks breakdown

Benchmark main polish-docs-again Change
256 KiB * 1000k chunk 183.6 ns 154.4 ns +18.88%
256 KiB * 100k chunk 183.6 ns 154.4 ns +18.88%
256 KiB * 10k chunk 183.6 ns 154.4 ns +18.88%
256 KiB * 1k chunk 212.8 ns 183.6 ns +15.89%

# for free to join this conversation on GitHub. Already have an account? # to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants