CI: Add frontend lint GitHub Actions workflow (#101559)
* lint frontend * set frontend as owners of the lint workflow * Fix syntax * fix warning * fix quote
This commit is contained in:
@@ -818,6 +818,7 @@ embed.go @grafana/grafana-as-code
|
|||||||
/.github/workflows/changelog.yml @zserge
|
/.github/workflows/changelog.yml @zserge
|
||||||
/.github/workflows/actions/changelog @zserge
|
/.github/workflows/actions/changelog @zserge
|
||||||
/.github/workflows/frontend-unit-tests.yml @grafana/grafana-frontend-platform
|
/.github/workflows/frontend-unit-tests.yml @grafana/grafana-frontend-platform
|
||||||
|
/.github/workflows/frontend-lint.yml @grafana/grafana-frontend-platform
|
||||||
|
|
||||||
# Generated files not requiring owner approval
|
# Generated files not requiring owner approval
|
||||||
/packages/grafana-data/src/types/featureToggles.gen.ts @grafanabot
|
/packages/grafana-data/src/types/featureToggles.gen.ts @grafanabot
|
||||||
|
|||||||
@@ -0,0 +1,55 @@
|
|||||||
|
name: Lint Frontend
|
||||||
|
on:
|
||||||
|
pull_request:
|
||||||
|
push:
|
||||||
|
branches:
|
||||||
|
- main
|
||||||
|
- release-*.*.*
|
||||||
|
|
||||||
|
jobs:
|
||||||
|
verify-i18n:
|
||||||
|
name: Verify i18n
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v4
|
||||||
|
- uses: actions/setup-node@v4
|
||||||
|
with:
|
||||||
|
node-version-file: '.nvmrc'
|
||||||
|
cache: 'yarn'
|
||||||
|
cache-dependency-path: 'yarn.lock'
|
||||||
|
- run: yarn install --immutable --check-cache
|
||||||
|
- run: |
|
||||||
|
extract_error_message='::error::Extraction failed. Make sure that you have no dynamic translation phrases, such as "t(`preferences.theme.{themeID}`, themeName)" and that no translation key is used twice. Search the output for '[warning]' to find the offending file.'
|
||||||
|
make i18n-extract || (echo "${extract_error_message}" && false)
|
||||||
|
- run: |
|
||||||
|
uncommited_error_message="::error::Translation extraction has not been committed. Please run 'make i18n-extract', commit the changes and push again."
|
||||||
|
file_diff=$(git diff --dirstat public/locales)
|
||||||
|
if [ -n "$file_diff" ]; then
|
||||||
|
echo $file_diff
|
||||||
|
echo "${uncommited_error_message}"
|
||||||
|
exit 1
|
||||||
|
fi
|
||||||
|
prettier:
|
||||||
|
name: Prettier
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v4
|
||||||
|
- uses: actions/setup-node@v4
|
||||||
|
with:
|
||||||
|
node-version-file: '.nvmrc'
|
||||||
|
cache: 'yarn'
|
||||||
|
cache-dependency-path: 'yarn.lock'
|
||||||
|
- run: yarn install --immutable --check-cache
|
||||||
|
- run: yarn run prettier:check
|
||||||
|
typecheck:
|
||||||
|
name: Typecheck
|
||||||
|
runs-on: ubuntu-latest
|
||||||
|
steps:
|
||||||
|
- uses: actions/checkout@v4
|
||||||
|
- uses: actions/setup-node@v4
|
||||||
|
with:
|
||||||
|
node-version-file: '.nvmrc'
|
||||||
|
cache: 'yarn'
|
||||||
|
cache-dependency-path: 'yarn.lock'
|
||||||
|
- run: yarn install --immutable --check-cache
|
||||||
|
- run: yarn run typecheck
|
||||||
Reference in New Issue
Block a user