-
Notifications
You must be signed in to change notification settings - Fork 5.6k
/
Dangerfile
42 lines (35 loc) · 2.04 KB
/
Dangerfile
1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
# We generally try to avoid big PRs
warn("Big PR") if git.lines_of_code > 500
# Show a warning for PRs that are Work In Progress
if (github.pr_body + github.pr_title).include?("WIP")
warn("Pull Request is Work in Progress")
end
# Contributors should always provide a changelog when submitting a PR
if github.pr_body.length < 5
warn("Please provide a changelog summary in the Pull Request description @#{github.pr_author}")
end
if git.modified_files.include?("snapshot/lib/assets/SnapshotHelper.swift")
warn("You modified `SnapshotHelper.swift`, make sure to update the version number at the bottom of the file to notify users about the new helper file.")
end
if git.modified_files.include?("snapshot/lib/assets/SnapshotHelperXcode8.swift")
warn("You modified `SnapshotHelperXcode8.swift`, make sure to update the version number at the bottom of the file to notify users about the new helper file.")
end
# To avoid "PR & Runs" for which tests don't pass, we want to make spec errors more visible
# The code below will run on Circle, parses the results in JSON and posts them to the PR as comment
containing_dir = ENV["CIRCLE_TEST_REPORTS"] || "." # for local testing
file_path = File.join(containing_dir, "rspec", "fastlane-junit-results.xml")
if File.exist?(file_path)
junit.parse(file_path)
junit.headers = [:name, :file]
junit.report
else
puts("Couldn't find any test artifacts in path #{file_path}")
end
# PRs being made on a branch from a different owner should warn to allow maintainers access to modify
head_owner = github.pr_json["head"]["repo"]["owner"]["login"]
base_owner = github.pr_json["base"]["repo"]["owner"]["login"]
if !github.pr_json["maintainer_can_modify"] && head_owner != base_owner
warn("If you would allow the maintainers access to make changes to your branch that would be 💯 " \
"This allows maintainers to help move pull requests through quicker if there are any changes that they can help with 😊 " \
"See more info at https://help.github.com/en/articles/allowing-changes-to-a-pull-request-branch-created-from-a-fork")
end