-
Notifications
You must be signed in to change notification settings - Fork 73
Closed
Labels
bugSomething isn't workingSomething isn't workinggood first issueGood for newcomersGood for newcomers
Description
I observed that bazel-diff does not consider the target changed if one of the target's input files gets deleted if the file was empty to begin with.
It seems that the hashing logic below produces the same result :
bazel-diff/cli/src/main/kotlin/com/bazel_diff/hash/SourceFileHasher.kt
Lines 91 to 105 in 4e72550
| } else { | |
| val absoluteFilePath = workingDirectory.resolve(filenamePath) | |
| val file = absoluteFilePath.toFile() | |
| if (file.exists()) { | |
| if (file.isFile) { | |
| if (modifiedFilepaths.isEmpty()) { | |
| putFile(file) | |
| } else if (modifiedFilepaths.any { workingDirectory.resolve(it) == absoluteFilePath }) { | |
| putFile(file) | |
| } | |
| } | |
| } else { | |
| logger.w { "File $absoluteFilePath not found" } | |
| } | |
| } |
Metadata
Metadata
Assignees
Labels
bugSomething isn't workingSomething isn't workinggood first issueGood for newcomersGood for newcomers