From 0062bc0fdd1edaa6b63be01dd778fd30514d3aaa Mon Sep 17 00:00:00 2001 From: Manav Rathi Date: Mon, 4 Mar 2024 11:50:04 +0530 Subject: [PATCH] Add Flutter linter for mobile The older version seemed to be disabled (it was getting triggered on a non-existent master branch), so I'm not sure if we want this on right now. But let's give it a go, will remove if it this needs some cleanup first. --- .github/workflows/mobile-lint.yml | 19 +++++++++++++ mobile/.github/workflows/code_quality.yml | 33 ----------------------- 2 files changed, 19 insertions(+), 33 deletions(-) create mode 100644 .github/workflows/mobile-lint.yml delete mode 100644 mobile/.github/workflows/code_quality.yml diff --git a/.github/workflows/mobile-lint.yml b/.github/workflows/mobile-lint.yml new file mode 100644 index 0000000000..289403e946 --- /dev/null +++ b/.github/workflows/mobile-lint.yml @@ -0,0 +1,19 @@ +name: "Lint (mobile)" + +on: + # Run on every push (this also covers pull requests) + push: + # But only if something changes inside mobile + paths: + - "mobile/**" + +jobs: + lint: + runs-on: ubuntu-latest + steps: + - uses: actions/checkout@v4 + with: + submodules: recursive + - run: flutter pub get + - run: flutter analyze --no-fatal-infos + - run: flutter test diff --git a/mobile/.github/workflows/code_quality.yml b/mobile/.github/workflows/code_quality.yml deleted file mode 100644 index c3b654ff20..0000000000 --- a/mobile/.github/workflows/code_quality.yml +++ /dev/null @@ -1,33 +0,0 @@ -name: Check Linter Rules -on: - pull_request: - types: [review_requested] - branches: - - master -jobs: - test: - name: Check the source code - runs-on: ubuntu-latest - steps: - - uses: actions/checkout@v3 - - uses: actions/cache@v2 - with: - path: ${{ runner.tool_cache }}/flutter - key: flutter-3.0.0-stable - # Setup the flutter environment. - - uses: subosito/flutter-action@v2.3.0 - with: - channel: 'stable' - flutter-version: '3.0.0' - - # Fetch sub modules - - run: git submodule update --init --recursive - - # Get flutter dependencies. - - name: Install packages - run: flutter pub get - - - name: Run Linter - run: flutter analyze --no-fatal-infos - - name: Run Test - run: flutter test