Skip to content
This repository has been archived by the owner on Sep 11, 2020. It is now read-only.

plumbing: format/commitgraph, add APIs for reading and writing commit-graph files #1128

Merged
merged 5 commits into from
Apr 24, 2019

Conversation

filipnavara
Copy link
Contributor

@filipnavara filipnavara commented Apr 22, 2019

Contributes to #965. This implements the low-level API for reading and writing the commit-graph files. Test cases are submitted in src-d/go-git-fixtures#16.

At the moment it doesn't allow adding custom chunks to the commit-graph file or reading them. The specification allows it and it should probably be supported since I used it for experiments before. However, Git itself doesn't write any non-standard chunks as of version 2.21.

@filipnavara filipnavara force-pushed the commitgraph-fmt branch 2 times, most recently from c315339 to c95d1f6 Compare April 23, 2019 13:22
@filipnavara
Copy link
Contributor Author

filipnavara commented Apr 23, 2019

How can I make the CI pick up more recent go-git-fixtures? I'm not sure whether go.mod is still used since newer Go ignores it by default and it was not updated for ages in go-git repository.

The actual coverage is 81.7% of statements, above the go-git average. Only error paths are not fully tested.

@mcuadros
Copy link
Contributor

#1131

…-graph files

Signed-off-by: Filip Navara <filip.navara@gmail.com>
Signed-off-by: Filip Navara <filip.navara@gmail.com>
Signed-off-by: Filip Navara <filip.navara@gmail.com>
Signed-off-by: Filip Navara <filip.navara@gmail.com>
…coded file binary identical

Signed-off-by: Filip Navara <filip.navara@gmail.com>
@filipnavara
Copy link
Contributor Author

Thanks, I rebased it and now it will hopefully pick up the new fixtures on CI.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants