-
Notifications
You must be signed in to change notification settings - Fork 7
Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
- Loading branch information
Florian Renaud
committed
Aug 23, 2022
1 parent
4388396
commit 7bb09c9
Showing
17 changed files
with
78 additions
and
345 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -1,4 +1,3 @@ | ||
|
|
||
| name: Code Quality Checks | ||
|
|
||
| on: | ||
|
|
@@ -62,55 +61,12 @@ jobs: | |
| - name: Prepare Danger | ||
| if: always() | ||
| run: | | ||
| <<<<<<< HEAD | ||
| results="$(cat */*/build/reports/ktlint/ktlint*/ktlint*.txt */build/reports/ktlint/ktlint*/ktlint*.txt | sed -r "s/\x1B\[([0-9]{1,3}(;[0-9]{1,2})?)?[mGK]//g")" | ||
| if [ -z "$results" ]; then | ||
| echo "::set-output name=add_comment::false" | ||
| else | ||
| body="👎\`Failed${results}\`" | ||
| body="${body//'%'/'%25'}" | ||
| body="${body//$'\n'/'%0A'}" | ||
| body="${body//$'\r'/'%0D'}" | ||
| body="$( echo $body | sed 's/\/home\/runner\/work\/tchap-android\/tchap-android\//\`<br\/>\`/g')" | ||
| body="$( echo $body | sed 's/\/src\/main\/java\// 🔸 /g')" | ||
| body="$( echo $body | sed 's/im\/vector\/app\///g')" | ||
| body="$( echo $body | sed 's/im\/vector\/lib\/attachmentviewer\///g')" | ||
| body="$( echo $body | sed 's/im\/vector\/lib\/multipicker\///g')" | ||
| body="$( echo $body | sed 's/im\/vector\/lib\///g')" | ||
| body="$( echo $body | sed 's/org\/matrix\/android\/sdk\///g')" | ||
| body="$( echo $body | sed 's/\/src\/androidTest\/java\// 🔸 /g')" | ||
| echo "::set-output name=add_comment::true" | ||
| echo "::set-output name=body::$body" | ||
| fi | ||
| - name: Find Comment | ||
| if: always() && github.event_name == 'pull_request' | ||
| uses: peter-evans/find-comment@v2 | ||
| id: fc | ||
| with: | ||
| issue-number: ${{ github.event.pull_request.number }} | ||
| comment-author: 'github-actions[bot]' | ||
| body-includes: Ktlint Results | ||
| - name: Add comment if needed | ||
| if: always() && github.event_name == 'pull_request' && steps.ktlint-results.outputs.add_comment == 'true' | ||
| uses: peter-evans/create-or-update-comment@v2 | ||
| with: | ||
| comment-id: ${{ steps.fc.outputs.comment-id }} | ||
| issue-number: ${{ github.event.pull_request.number }} | ||
| body: | | ||
| ### Ktlint Results | ||
| ${{ steps.ktlint-results.outputs.body }} | ||
| edit-mode: replace | ||
| - name: Delete comment if needed | ||
| if: always() && github.event_name == 'pull_request' && steps.fc.outputs.comment-id != '' && steps.ktlint-results.outputs.add_comment == 'false' | ||
| uses: actions/github-script@v3 | ||
| ======= | ||
| npm install --save-dev @babel/core | ||
| npm install --save-dev @babel/plugin-transform-flow-strip-types | ||
| yarn add danger-plugin-lint-report --dev | ||
| - name: Danger lint | ||
| if: always() | ||
| uses: danger/[email protected] | ||
| >>>>>>> v1.4.32 | ||
| with: | ||
| args: "--dangerfile tools/danger/dangerfile-lint.js" | ||
| env: | ||
|
|
@@ -134,110 +90,3 @@ jobs: | |
| with: | ||
| name: dependency-analysis | ||
| path: build/reports/dependency-check-report.html | ||
| <<<<<<< HEAD | ||
|
|
||
| # Lint for main module | ||
| android-lint: | ||
| name: Android Linter | ||
| runs-on: ubuntu-latest | ||
| # Allow all jobs on main and develop. Just one per PR. | ||
| concurrency: | ||
| group: ${{ github.ref == 'refs/heads/main' && format('android-lint-main-{0}', github.sha) || github.ref == 'refs/heads/develop' && format('android-lint-develop-{0}', github.sha) || format('android-lint-{0}', github.ref) }} | ||
| cancel-in-progress: true | ||
| steps: | ||
| - uses: actions/checkout@v3 | ||
| - uses: actions/cache@v3 | ||
| with: | ||
| path: | | ||
| ~/.gradle/caches | ||
| ~/.gradle/wrapper | ||
| key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle*', '**/gradle-wrapper.properties') }} | ||
| restore-keys: | | ||
| ${{ runner.os }}-gradle- | ||
| - name: Lint analysis | ||
| run: ./gradlew clean :vector:lint --stacktrace $CI_GRADLE_ARG_PROPERTIES | ||
| - name: Upload reports | ||
| if: always() | ||
| uses: actions/upload-artifact@v3 | ||
| with: | ||
| name: lint-report | ||
| path: | | ||
| vector/build/reports/*.* | ||
| # Lint for Gplay and Fdroid release APK | ||
| apk-lint: | ||
| name: Lint APK (${{ matrix.target }}) | ||
| runs-on: ubuntu-latest | ||
| if: github.ref != 'refs/heads/main' | ||
| strategy: | ||
| fail-fast: false | ||
| matrix: | ||
| target: [ GplayBtchapWithoutvoipWithpinning, FdroidBtchapWithoutvoipWithoutpinning ] | ||
| # Allow all jobs on develop. Just one per PR. | ||
| concurrency: | ||
| group: ${{ github.ref == 'refs/heads/develop' && format('apk-lint-develop-{0}-{1}', matrix.target, github.sha) || format('apk-lint-{0}-{1}', matrix.target, github.ref) }} | ||
| cancel-in-progress: true | ||
| steps: | ||
| - uses: actions/checkout@v3 | ||
| - uses: actions/cache@v3 | ||
| with: | ||
| path: | | ||
| ~/.gradle/caches | ||
| ~/.gradle/wrapper | ||
| key: ${{ runner.os }}-gradle-${{ hashFiles('**/*.gradle*', '**/gradle-wrapper.properties') }} | ||
| restore-keys: | | ||
| ${{ runner.os }}-gradle- | ||
| - name: Lint ${{ matrix.target }} release | ||
| run: ./gradlew clean lint${{ matrix.target }}Release --stacktrace $CI_GRADLE_ARG_PROPERTIES | ||
| - name: Upload ${{ matrix.target }} linting report | ||
| if: always() | ||
| uses: actions/upload-artifact@v3 | ||
| with: | ||
| name: release-lint-report-${{ matrix.target }} | ||
| path: | | ||
| vector/build/reports/*.* | ||
| detekt: | ||
| name: Detekt Analysis | ||
| runs-on: ubuntu-latest | ||
| # Allow all jobs on main and develop. Just one per PR. | ||
| concurrency: | ||
| group: ${{ github.ref == 'refs/heads/main' && format('detekt-main-{0}', github.sha) || github.ref == 'refs/heads/develop' && format('detekt-develop-{0}', github.sha) || format('detekt-{0}', github.ref) }} | ||
| cancel-in-progress: true | ||
| steps: | ||
| - uses: actions/checkout@v3 | ||
| - name: Run detekt | ||
| run: | | ||
| ./gradlew detekt $CI_GRADLE_ARG_PROPERTIES | ||
| - name: Upload reports | ||
| if: always() | ||
| uses: actions/upload-artifact@v3 | ||
| with: | ||
| name: detekt-report | ||
| path: | | ||
| */build/reports/detekt/detekt.html | ||
| # towncrier: | ||
| # name: Towncrier check | ||
| # runs-on: ubuntu-latest | ||
| # if: github.event_name == 'pull_request' && github.head_ref == 'develop' | ||
| # steps: | ||
| # - uses: actions/checkout@v3 | ||
| # - name: Set up Python 3.8 | ||
| # uses: actions/setup-python@v4 | ||
| # with: | ||
| # python-version: 3.8 | ||
| # - name: Install towncrier | ||
| # run: | | ||
| # python3 -m pip install towncrier | ||
| # - name: Run towncrier | ||
| # # Fetch the pull request' base branch so towncrier will be able to | ||
| # # compare the current branch with the base branch. | ||
| # # Source: https://github.com/actions/checkout/#fetch-all-branches. | ||
| # run: | | ||
| # git fetch --no-tags origin +refs/heads/${BASE_BRANCH}:refs/remotes/origin/${BASE_BRANCH} | ||
| # towncrier check --compare-with origin/${BASE_BRANCH} | ||
| # env: | ||
| # BASE_BRANCH: ${{ github.base_ref }} | ||
| ======= | ||
| >>>>>>> v1.4.32 | ||
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.