search

clehner / abstract-pull-git-repo

abstract interface for git repos using pull streams
https://git.scuttlebot.io/%25a%2FAmkaq7UGRD7%2FJiov%2F1iMeT2OZfeAY3p7mk%2BkUYn6k%3D.sha256
6 stars 1 forks source link

Tests for packfiles #1

Open axic opened 8 years ago

axic commented 8 years ago

Can you add canonical tests for packfiles?

clehner commented 8 years ago

I agree that tests should be added for the packfile-related methods. However, I do not have immediate plans to add them. Are you looking to build a new pull-git-repo implementation?

axic commented 8 years ago

I did a PoC for using Ethereum+IPFS/Swarm as a backend: https://github.com/axic/git-remote-mango. Code quality is that of a hackathon 😃

clehner commented 8 years ago

Nice! I would think that since IPFS does data deduplication, packfiles might not be necessary.

axic commented 8 years ago

For IPFS, conceptually, the objects are much better. Due to speed concerns and if the Git repo is only used for archival purposes (i.e. it is only pushed at every release, while the main development is on a different remote) using a packfile would be advantageous.

I plan to move on to Swarm, where packfiles wouldn't be needed at all.

I still think it would be good to have tests for the future :)