Skip to content

Search backend: do not error on empty repo

Warren Gifford requested to merge cc/fix-empty-repo-commit-after into main

Created by: camdencheek

The recent changes to how repo:has.commit.after() executes made it so we no longer ignore errors. However, in the case of repo:has.commit.after(), we will receive an error for empty repos which should be interpreted as "repo does not have commit after."

This updates the logic to do so.

Related Slack thread here

Test plan

Added unit tests and manually tested that repo:has.commit.after() does not bomb out on empty repos now.

Merge request reports