aai-institute / lakefs-spec

An fsspec implementation for the lakeFS project
http://lakefs-spec.org/
Apache License 2.0
39 stars 4 forks source link

Fix transaction doc on `Placeholder` values #226

Closed nicholasjng closed 9 months ago

nicholasjng commented 9 months ago

The newest PR has made the use of Placeholder.unwrap() obsolete, so the value can be reused as is, with the same semantics.

Changes the paragraph and code example accordingly to reflect this fact.

codecov[bot] commented 9 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (90f76a9) 90.56% compared to head (d212ef7) 90.56%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #226 +/- ## ======================================= Coverage 90.56% 90.56% ======================================= Files 7 7 Lines 551 551 Branches 93 93 ======================================= Hits 499 499 Misses 27 27 Partials 25 25 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.