Merge pull request #2604 from krille-chan/krille/check-for-commented-code
build: Check for commented code in CI
This commit is contained in:
commit
67d9eaf5b7
2 changed files with 10 additions and 5 deletions
13
.github/workflows/integrate.yaml
vendored
13
.github/workflows/integrate.yaml
vendored
|
|
@ -25,13 +25,20 @@ jobs:
|
||||||
- name: Check license compliance
|
- name: Check license compliance
|
||||||
run: dart run license_checker check-licenses -c licenses.yaml --problematic
|
run: dart run license_checker check-licenses -c licenses.yaml --problematic
|
||||||
- run: flutter analyze
|
- run: flutter analyze
|
||||||
|
- run: dart run dart_code_linter:metrics check-unused-code lib
|
||||||
|
- run: dart run dart_code_linter:metrics check-unused-files lib
|
||||||
|
- run: dart run dart_code_linter:metrics check-unused-l10n lib
|
||||||
|
- name: Check for commented-out Dart code with semicolons
|
||||||
|
run: |
|
||||||
|
if grep -R --include="*.dart" -nE '^[[:space:]]*//[^/<].*;[[:space:]]*$' lib/; then
|
||||||
|
echo ""
|
||||||
|
echo "❌ Found commented-out Dart code ending with semicolon."
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
- name: Add Firebase Messaging
|
- name: Add Firebase Messaging
|
||||||
run: ./scripts/add-firebase-messaging.sh
|
run: ./scripts/add-firebase-messaging.sh
|
||||||
- run: flutter analyze
|
- run: flutter analyze
|
||||||
- run: flutter test
|
- run: flutter test
|
||||||
- run: dart run dart_code_linter:metrics check-unused-code lib
|
|
||||||
- run: dart run dart_code_linter:metrics check-unused-files lib
|
|
||||||
- run: dart run dart_code_linter:metrics check-unused-l10n lib
|
|
||||||
|
|
||||||
build_debug_apk:
|
build_debug_apk:
|
||||||
runs-on: ubuntu-latest
|
runs-on: ubuntu-latest
|
||||||
|
|
|
||||||
|
|
@ -32,8 +32,6 @@ import '../utils/account_bundles.dart';
|
||||||
import '../utils/background_push.dart';
|
import '../utils/background_push.dart';
|
||||||
import 'local_notifications_extension.dart';
|
import 'local_notifications_extension.dart';
|
||||||
|
|
||||||
// import 'package:flutter_secure_storage/flutter_secure_storage.dart';
|
|
||||||
|
|
||||||
class Matrix extends StatefulWidget {
|
class Matrix extends StatefulWidget {
|
||||||
final Widget? child;
|
final Widget? child;
|
||||||
|
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue