Add optional support for `camino::Utf8PathBuf`
Nobody has claimed this yet.
- Dominant language
- Rust
- Stars
- 9.9k
- Forks
- 734
- Avg merge
- 6h 36m
- Merged PRs (30d)
- 8
Description
Motivation
I would like to use camino::Utf8PathBuf as a column type.
Proposed Solutions
Implement the relevant sea-orm traits for Utf8PathBuf, gated behind a new feature with-camino.
Since Utf8PathBuf supports infallible conversion to and from std::string::String, I think it's okay to back it with such. Therefore the implementation should be rather trivial.
Additional Information
I have previously submitted a PR for camino (it was some time ago; I got distracted by other obligations), but as its maintainer correctly pointed out, these impls should belong in sea-orm instead.
I am willing and able to work on a PR for this, but I'm submitting this issue first to get any potential concerns and/or blockers sorted.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating existing optional integrations for column types and the SeaORM traits used for String-backed values. Check how feature flags and integration tests are organized, then verify that enabling with-camino supports camino::Utf8PathBuf as a column type and that the relevant conversions pass.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- rust
- Domain
- database
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 48/100