1
95%
master: 95%

Ran 10 Apr 2025 10:15PM UTC

Files 5

Run time 0s

Badge

Embed ▾

Committed 10 Apr 2025 10:14PM UTC coverage: 94.048% (+0.05%) from 93.994%

Job # 14391306314.1

Build Type

push

github

Committed by

cljoly

Commit Message

refactor: Keep migrations in `Cow` instead of `Vec`

This builds on https://github.com/cljoly/rusqlite_migration/pull/251 and
proposes another path. Like #251, it provides a method to store the
migrations in a `const` context, but:
* it does so without breaking changes, at least compared to the latest
  release: `new` is not const anymore, but that change was not released
  in a stable version.
* exposes only concrete types like `Vec` and slices to build the
  migrations, instead of the discussed `impl Into<Cow<'m, [M<'m>]>>`.
  This should make the compiler report clearer errors, improving
  usability.
* we can still easily built from a `Cow` type, by borrowing it.

Drawbacks
---------

The main drawback I can think of is this: the most intuitive function to
call (`new`) takes a `Vec`, while it would be better to just use a slice
(which `from_slice` takes). But that cost should be minimal and
preserves backward compatibility.

Co-authored-by: Joaquim Monteiro <joaquim.monteiro@protonmail.com>

Run Details

316 of 336 relevant lines covered (94.05%)

5.71 hits per line

cljoly / rusqlite_migration / 14391306314 / 1
95%
master: 95%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 14391306314.1

cljoly / rusqlite_migration / 14391306314 / 1 95% master: 95%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Source Files on job 14391306314.1

cljoly / rusqlite_migration / 14391306314 / 1
95%
master: 95%

README BADGES
x