8000 plumbing: format/commitgraph, add APIs for reading and writing commit-graph files by filipnavara · Pull Request #1128 · src-d/go-git · GitHub
[go: up one dir, main page]

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

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

0