Skip to content

Use new topo-order logic with GIT_TEST_COMMIT_GRAPH

Derrick Stolee via GitGitGadget git 1 version In review

Get series with b4
b4 am [email protected]

Versions newest first

  1. v1 Latest1 patch 2 repliesIn review

Patches in v1 No applies recorded yet

#Subject and filesSizeRepliesApplied
1/1revision.c: use new topo-order logic in testsrevision.c+4 -02 replies

Reviews trailers per patch

No Reviewed-by, Acked-by or Tested-by trailers for v1 yet.

Read the v1 thread

Cover letter [PATCH 0/1] Use new topo-order logic with GIT_TEST_COMMIT_GRAPH

The recent Git test report for v2.20.0-rc0 shows that the logic around UNINTERESTING commits is not covered by the test suite. This is because the code is currently unreachable! See the commit message for details.

An alternate approach would be to delete the code around UNINTERESTING commits, but that doesn't seem necessary.

Thanks, -Stolee

Derrick Stolee (1):
revision.c: use new topo-order logic in tests

 revision.c | 4 ++++
 1 file changed, 4 insertions(+)

base-commit: 561b583749b7428f1790f03164d0d0e75be71d7b
Published-As: https://github.com/gitgitgadget/git/releases/tags/pr-83%2Fderrickstolee%2Ftopo-order-test-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-83/derrickstolee/topo-order-test-v1
Pull-Request: https://github.com/gitgitgadget/git/pull/83

Signature · 1 line
gitgitgadget