Skip to content

feat: expose PathBufWrap utility for public access#3694

Merged
JohnTitor merged 3 commits intoactix:mainfrom
imgurbot12:feat/pathbuf
Feb 2, 2026
Merged

feat: expose PathBufWrap utility for public access#3694
JohnTitor merged 3 commits intoactix:mainfrom
imgurbot12:feat/pathbuf

Conversation

@imgurbot12
Copy link
Contributor

@imgurbot12 imgurbot12 commented Jul 15, 2025

PR Type

Feature / Refactor

PR Checklist

  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • A changelog entry has been made for the appropriate packages.
  • Format code with the latest stable rustfmt.
  • (Team) Label with affected crates and semver status.

Overview

Hello! I've been writing a lot of services for actix-web recently and found that I've been reusing the code written for PathBufWrap a lot.

So far I've been copy-pasting it around, but I think it might be better for me and others like me that really appreciate this little utility if it was exported from the library so it could just be used directly.

@robjtede robjtede added A-files project: actix-files B-semver-minor labels Jul 15, 2025
Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

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

I've tweaked methods of PathBufWrap but overall LGTM. Thanks!

@JohnTitor JohnTitor added this pull request to the merge queue Feb 2, 2026
Merged via the queue into actix:main with commit 5ee6b59 Feb 2, 2026
14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-files project: actix-files B-semver-minor

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants