Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use --fast instead of --relaxed #13

Merged
merged 1 commit into from
Apr 15, 2024
Merged

Use --fast instead of --relaxed #13

merged 1 commit into from
Apr 15, 2024

Conversation

matrss
Copy link
Owner

@matrss matrss commented Apr 15, 2024

This change is in anticipation of the VURL backend and the --verifiable option of git annex addurl, which would allow automatic checksum verification for every download after the first one, without needing a backend migration. The documentation of --verifiable states that it will only verify subsequent downloads against the recorded checksum when used with --fast, while a combination with --relaxed would always accept the content it gets from the web / special remote.

This change is in anticipation of the VURL backend and the --verifiable
option of git annex addurl, which would allow automatic checksum
verification for every download after the first one, without needing a
backend migration. The documentation of --verifiable states that it will
only verify subsequent downloads against the recorded checksum when used
with --fast, while a combination with --relaxed would always accept the
content it gets from the web / special remote.
Copy link

codecov bot commented Apr 15, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 76.87%. Comparing base (4f3e997) to head (d28be16).

Additional details and impacted files
@@           Coverage Diff           @@
##             main      #13   +/-   ##
=======================================
  Coverage   76.87%   76.87%           
=======================================
  Files           5        5           
  Lines         173      173           
=======================================
  Hits          133      133           
  Misses         40       40           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@matrss matrss merged commit af3d770 into main Apr 15, 2024
8 of 9 checks passed
@matrss matrss deleted the replace-relaxed-with-fast branch April 15, 2024 09:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant