Skip to content

Commit 88412e3

Browse files
committed
[build] update names for clarity
1 parent 0c2aff1 commit 88412e3

1 file changed

Lines changed: 7 additions & 7 deletions

File tree

rake_tasks/java.rake

Lines changed: 7 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -53,19 +53,19 @@ def verify_java_release_targets
5353
current_targets = output.lines.map(&:strip).reject(&:empty?).select { |line| line.start_with?('//') }
5454
end
5555

56-
missing_targets = JAVA_RELEASE_TARGETS - current_targets
57-
extra_targets = current_targets - JAVA_RELEASE_TARGETS
56+
obsolete_targets = JAVA_RELEASE_TARGETS - current_targets
57+
unlisted_targets = current_targets - JAVA_RELEASE_TARGETS
5858

59-
return if missing_targets.empty? && extra_targets.empty?
59+
return if obsolete_targets.empty? && unlisted_targets.empty?
6060

6161
error_message = 'Java release targets are out of sync with Bazel query results.'
6262

63-
unless missing_targets.empty?
64-
error_message += "\nObsolete targets (in list but not in Bazel): #{missing_targets.join(', ')}"
63+
unless obsolete_targets.empty?
64+
error_message += "\nObsolete targets (in list but not in Bazel): #{obsolete_targets.join(', ')}"
6565
end
6666

67-
unless extra_targets.empty?
68-
error_message += "\nMissing targets (in Bazel but not in list): #{extra_targets.join(', ')}"
67+
unless unlisted_targets.empty?
68+
error_message += "\nMissing targets (in Bazel but not in list): #{unlisted_targets.join(', ')}"
6969
end
7070

7171
raise error_message

0 commit comments

Comments
 (0)