Skip to content

codeintel: Bulk resolve commits in dependencies service

Warren Gifford requested to merge ef/31639-resolve-commits into main

Created by: efritz

This PR adds an additional step resolveRepoCommits step to dependency resolution. As part of #31639 (closed), we will need to ensure that we have a canonical hash for symbolic names persisted in the database. This is a pre-step that makes that data available in the lockfileDependencies method.

This implementation is very similar to CommitsExist elsewhere in the codeintel codebase.

Test plan

Existing (modified) service unit test as well as existing unit tests in the vcs package.

Merge request reports