search

tahoe-lafs / magic-folder

Tahoe-LAFS-based file synchronization
Other
25 stars 7 forks source link

702.document snapshots #710

Closed meejah closed 1 year ago

meejah commented 1 year ago

Fixes #702

codecov[bot] commented 1 year ago

Codecov Report

Merging #710 (5e6883f) into main (a68f9c7) will not change coverage. The diff coverage is n/a.

@@           Coverage Diff           @@
##             main     #710   +/-   ##
=======================================
  Coverage   91.79%   91.79%           
=======================================
  Files          38       38           
  Lines        5144     5144           
  Branches      640      640           
=======================================
  Hits         4722     4722           
  Misses        277      277           
  Partials      145      145
Flag Coverage Δ
integration 72.97% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

:mega: We’re building smart automated test selection to slash your CI/CD build times. Learn more

meejah commented 1 year ago

Note to self: the metadata entries in the Collective and Personal should be @metadata instead (e.g. in all the diagrams).

meejah commented 1 year ago

Besides the above tweak needed for the diagrams, I think this is ready for some Opinions if you have any @exarkun (you can see a rendered version of this branch by clicking through the "read the docs" check).

meejah commented 1 year ago

I tried ensuring all the SVGs are "plain" (not "Inkscape") SVGs but github / firefox is still rendering the arrows w/o heads.

That's the only gnit though; I'm going to merge this -- please file followup tickets if anything is missing or unclear :)