Skip to content

Commit 937d9ca

Browse files
committed
fix: Avoid creating empty commits
1 parent 4c3e663 commit 937d9ca

File tree

1 file changed

+8
-4
lines changed
  • services/datalad/datalad_service/common

1 file changed

+8
-4
lines changed

services/datalad/datalad_service/common/git.py

Lines changed: 8 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -176,10 +176,14 @@ async def git_commit_index(
176176
committer = pygit2.Signature(COMMITTER_NAME, COMMITTER_EMAIL)
177177
if not author:
178178
author = committer
179-
if parents is None:
180-
parent_commits = [str(repo.head.target)]
181-
else:
182-
parent_commits = parents
179+
180+
parent_commits = [repo.head.target] if parents is None else parents
181+
if len(parent_commits) == 1 and not any(
182+
repo.index.diff_to_tree(repo.get(parent_commits[0]).tree)
183+
):
184+
# No changes to commit
185+
return
186+
183187
tree = repo.index.write_tree()
184188
commit = repo.create_commit(
185189
'refs/heads/main', author, committer, message, tree, parent_commits

0 commit comments

Comments
 (0)