Danger

.gitignore

@@ -73,3 +73,6 @@ fastlane/test_output
 
 ## AppCode
 .idea/
+
+## Ruby-related
+Gemfile.lock

.swiftlint.yml

@@ -11,7 +11,3 @@ included:
 type_body_length:
   - 400 #warning
   - 600 #error
-
-file_length:
-  - 800 #warning
-  - 1000 #error

Dangerfile

@@ -0,0 +1,17 @@
+# Lint all Swift files
+swiftlint.lint_files inline_mode: true
+
+# Make it more obvious that a PR is a work in progress and shouldn't be merged yet
+if github.pr_title.include? "[WIP]"
+    warn "PR is classed as Work in Progress"
+end
+
+# Warn when there is a big PR
+if git.lines_of_code > 500
+    warn "Big PR"
+end
+
+# Mainly to encourage writing up some reasoning about the PR, rather than just leaving a title
+if github.pr_body.length == 0
+  warn "Please provide a summary in the Pull Request description"
+end

GEMFILE

@@ -0,0 +1,4 @@
+source 'https://rubygems.org'
+
+gem 'danger'
+gem 'danger-swiftlint'

buddybuild_postclone.sh

@@ -2,8 +2,4 @@
 
 swift package generate-xcodeproj
 
-if ! which swiftlint >/dev/null; then
-    brew install swiftlint
-fi
-
-swiftlint
+brew upgrade swiftlint