RFR: 1854: Add "clean" label for merge PRs that have no merge resolution [v3]

Zhao Song zsong at openjdk.org
Wed Apr 12 23:22:55 UTC 2023


On Wed, 12 Apr 2023 23:19:23 GMT, Zhao Song <zsong at openjdk.org> wrote:

>> vcs/src/main/java/org/openjdk/skara/vcs/Repository.java line 186:
>> 
>>> 184:     }
>>> 185: 
>>> 186:     boolean isEmptyCommit(Hash hash);
>> 
>> We should add a javadoc comment describing the expected behavior for this method, specifically for merge commits.
>
> Sure. But I think it also works for simple empty commit(not merge commit)

At first, I name this method as `isEmptyMergeCommit` and later I thought it also works for simple empty commit, so I changed the method name to `isEmptyCommit`

-------------

PR Review Comment: https://git.openjdk.org/skara/pull/1500#discussion_r1164753601


More information about the skara-dev mailing list