Skip to content

Conversation

yarikoptic
Copy link
Member

TODOs

  • make git-annex invocations optional/autodetected based on presence of .git/annex
  • review git-annex behavior -- I think due to dotfiles. we might want some explicit setting to be tuned up in git annex config

Copy link

codecov bot commented Mar 5, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 82.48%. Comparing base (9e1e601) to head (29d52d8).

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #816   +/-   ##
=======================================
  Coverage   82.48%   82.48%           
=======================================
  Files          42       42           
  Lines        4323     4323           
=======================================
  Hits         3566     3566           
  Misses        757      757           

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

@yarikoptic
Copy link
Member Author

@claude review the script modified by this PR - its purpose is to maintain and update mapping for subject ids to new sequentially growing ids. Then

  • tune to use git-annex only if it is a git-annex repo (e.g. when there is .git/annex)
  • if file starts with period, make script ensure that annex.dotfiles is set to true (git annex config --set annex.dotfiles true )
  • add testing (could use e.g. bats or even code in python via pytest etc) that mapping happens as expected and file contains desired mapping

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