From 67749e75347ee7ae94ba6f6929846389d8e12118 Mon Sep 17 00:00:00 2001 From: Tom Hughes Date: Sat, 26 Oct 2024 12:17:38 +0100 Subject: [PATCH] Make danger labels use consisent capitalisation Existing labels are all lower case and avoid spaces. --- Dangerfile | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/Dangerfile b/Dangerfile index 3445f5705..3148bafb6 100644 --- a/Dangerfile +++ b/Dangerfile @@ -4,9 +4,9 @@ pr_number = github.pr_json["number"] # Report if number of changed lines is > 500 if git.lines_of_code > 500 warn("Number of updated lines of code is too large to be in one PR. Perhaps it should be separated into two or more?") - auto_label.set(pr_number, "Big PR", "FBCA04") + auto_label.set(pr_number, "big-pr", "FBCA04") else - auto_label.remove("Big PR") + auto_label.remove("big-pr") end # Get list of translation files (except en.yml) which are modified @@ -16,17 +16,17 @@ end # Report if some translation file (except en.yml) is modified if modified_yml_files.empty? - auto_label.remove("Inappropriate Translations") + auto_label.remove("inappropriate-translations") else modified_files_str = modified_yml_files.map { |file| "`#{file}`" }.join(", ") warn("The following YAML files other than `en.yml` have been modified: #{modified_files_str}. Only `en.yml` is allowed to be changed. Translations are updated via Translatewiki, see CONTRIBUTING.md.") - auto_label.set(pr_number, "Inappropriate Translations", "B60205") + auto_label.set(pr_number, "inappropriate-translations", "B60205") end # Report if there are merge-commits in PR if git.commits.any? { |c| c.parents.count > 1 } warn("Merge commits are found in PR. Please rebase to get rid of the merge commits in this PR, see CONTRIBUTING.md.") - auto_label.set(pr_number, "Merge Commits", "D93F0B") + auto_label.set(pr_number, "merge-commits", "D93F0B") else - auto_label.remove("Merge Commits") + auto_label.remove("merge-commits") end -- 2.39.5