Merge pull request #59 from gabrielbmoro/feature/try-add-danger-dart

Support for Danger

Author: gabrielbmoro

.github/workflows/danger.yml

@@ -2,14 +2,31 @@ name: Danger
 
 on:
   pull_request:
+    branches:
+      - main
 
 jobs:
-  build:
+  danger:
     runs-on: ubuntu-latest
-    name: "Run Danger"
+
     steps:
-      - uses: actions/checkout@v4
-      - name: Danger
-        uses: danger/kotlin@1.3.3
+      - name: Checkout repository
+        uses: actions/checkout@v4
+
+      - name: Set up Flutter
+        uses: ./.github/actions/setup-flutter
+
+      - name: Install dependencies
+        run: flutter pub get
+        shell: bash
+
+      - name: Install Danger-js
+        run: npm install -g danger
+    
+      - name: Activate command
+        run: flutter pub global activate danger_dart
+    
+      - name: Run danger ci
+        run: danger_dart ci
         env:
-          GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
+            GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}

.github/workflows/pr.yml

@@ -18,7 +18,7 @@ jobs:
 
       - name: Generate Resources
         run: flutter gen-l10n
-        
+
       - name: Run tests
         run: flutter test
 

dangerfile.dart

@@ -0,0 +1,29 @@
+import 'package:danger_core/danger_core.dart';
+
+const bigPRThreshold = 300;
+
+void main() {
+  // WIP check
+  if (danger.github.pr.title.contains('WIP')) {
+    warn('PR is considered WIP');
+  }
+
+  // Unit tests check
+  final allSourceFiles = [
+    ...danger.git.modifiedFiles,
+    ...danger.git.createdFiles,
+  ];
+
+  if (allSourceFiles.any((file) => file.endsWith('_test.dart'))) {
+    message('Thanks for updating the unit tests');
+  }
+
+  // Big PR check
+  final prAdditions = danger.github.pr.additions ?? 0;
+  final prDeletions = danger.github.pr.additions ?? 0;
+  final isBigPR = (prAdditions + prDeletions) > bigPRThreshold;
+
+  if (isBigPR) {
+    warn('Big PR, try to keep changes smaller if you can...');
+  }
+}

pubspec.lock

@@ -217,6 +217,14 @@ packages:
       url: "https://pub.dev"
     source: hosted
     version: "1.0.8"
+  danger_core:
+    dependency: "direct dev"
+    description:
+      name: danger_core
+      sha256: f949e126fcd0dcbd41ce973d733f2cb18426ee3870c0ec16ad331a2d9ce698c1
+      url: "https://pub.dev"
+    source: hosted
+    version: "2.0.0"
   dart_style:
     dependency: transitive
     description:

pubspec.yaml

@@ -62,6 +62,7 @@ dev_dependencies:
   injectable_generator: ^2.5.0
   build_runner: ^2.5.4
   auto_route_generator: ^10.1.0
+  danger_core: ^2.0.0
 
 # For information on the generic Dart part of this file, see the
 # following page: https://dart.dev/tools/pub/pubspec