Add static analysis for tsc --strict (#9212)
* Add static analysis for tsc --strict * Make tsconfig legal * Make tsconfig more legal * Update static_analysis.yaml * Test strict mode check * Revert "Test strict mode check" This reverts commit 879df9879738fabfa02466ce07634a0f5c132311.
This commit is contained in:
parent
fe409d992c
commit
e11b0cef47
2 changed files with 38 additions and 3 deletions
37
.github/workflows/static_analysis.yaml
vendored
37
.github/workflows/static_analysis.yaml
vendored
|
@ -90,5 +90,40 @@ jobs:
|
|||
|
||||
- name: Dead Code Analysis
|
||||
run: |
|
||||
cd element-web
|
||||
cd element-web
|
||||
yarn run analyse:unused-exports
|
||||
|
||||
tsc-strict:
|
||||
name: Typescript Strict Error Checker
|
||||
if: github.event_name == 'pull_request'
|
||||
runs-on: ubuntu-latest
|
||||
permissions:
|
||||
pull-requests: read
|
||||
checks: write
|
||||
steps:
|
||||
- uses: actions/checkout@v3
|
||||
|
||||
- name: Get diff lines
|
||||
id: diff
|
||||
uses: Equip-Collaboration/diff-line-numbers@v1.0.0
|
||||
with:
|
||||
include: '["\\.tsx?$"]'
|
||||
|
||||
- name: Detecting files changed
|
||||
id: files
|
||||
uses: futuratrepadeira/changed-files@v3.2.1
|
||||
with:
|
||||
repo-token: ${{ secrets.GITHUB_TOKEN }}
|
||||
pattern: '^.*\.tsx?$'
|
||||
|
||||
- uses: t3chguy/typescript-check-action@main
|
||||
with:
|
||||
repo-token: ${{ secrets.GITHUB_TOKEN }}
|
||||
use-check: false
|
||||
check-fail-mode: added
|
||||
output-behaviour: annotate
|
||||
ts-extra-args: '--strict'
|
||||
files-changed: ${{ steps.files.outputs.files_updated }}
|
||||
files-added: ${{ steps.files.outputs.files_created }}
|
||||
files-deleted: ${{ steps.files.outputs.files_deleted }}
|
||||
line-numbers: ${{ steps.diff.outputs.lineNumbers }}
|
||||
|
|
|
@ -17,13 +17,13 @@
|
|||
"es2020",
|
||||
"dom",
|
||||
"dom.iterable"
|
||||
],
|
||||
]
|
||||
},
|
||||
"include": [
|
||||
"./src/**/*.ts",
|
||||
"./src/**/*.tsx",
|
||||
"./test/**/*.ts",
|
||||
"./test/**/*.tsx",
|
||||
"./test/**/*.tsx"
|
||||
],
|
||||
"exclude": [
|
||||
"./test/end-to-end-tests/"
|
||||
|
|
Loading…
Reference in a new issue