-
Notifications
You must be signed in to change notification settings - Fork 0
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
Check how the best make use of git branches for palaeographic snapshot of the TEI corpus #65
Comments
Available operations using git:
Those two operations are very simple and safe. The second may require passing a lot of file names in case of a bulk update. If helpful, this could possibly be scripted. Important caveat to second operation: this is not a git merge, but a blunt overwrite. The series of commits from main, their comments, dates and authors will be lost by the flattening into a single overwrite. |
@simonastoyanova & @JonPrag I've tested the git solution above and I think it should work ok for your needs. The main caveat being that when the snapshot is updated (in bulk or not) the history of changes is not ported to that branch. The changes coming from the editorial branch are flattened without metadata. But that should be acceptable if the snapshot is never edited manually and the files in the main branch contain last modification dates or a version number. |
That works for me. The changes will be traceable through the main branch if someone wanted to double check a new reading. The snapshot won't be edited manually, if I correct a reading based on palaeographic analysis, I will go change the original file in the main branch and port it back to the snapshot (don't expect this to happen often but it has once already, so useful as a scenario). |
Some further comments after our meeting today:
In terms of changes being ported from the main branch to the palaeo branch, the hashing system should work with some testing as refinement; to recognise change of id, to recognise change of token etc., to be decided and tried. |
Requirements:
The text was updated successfully, but these errors were encountered: