chore: Remove non-blocking behavior from linting and type checking in CI workflow
All checks were successful
CI / skip-ci-check (pull_request) Successful in 1m35s
CI / lint-and-type-check (pull_request) Successful in 2m11s
CI / python-lint (pull_request) Successful in 2m0s
CI / test-backend (pull_request) Successful in 3m42s
CI / build (pull_request) Successful in 4m43s
CI / secret-scanning (pull_request) Successful in 1m44s
CI / dependency-scan (pull_request) Successful in 1m41s
CI / sast-scan (pull_request) Successful in 2m46s
CI / workflow-summary (pull_request) Successful in 1m34s

This commit updates the CI workflow to remove the `|| true` command from the linting and type checking steps, ensuring that these checks will fail the build process if issues are encountered. This change enforces stricter quality control in the CI pipeline, requiring developers to address linting and type checking errors before proceeding with the build.
This commit is contained in:
Tanya 2026-01-12 13:00:01 -05:00
parent 0e673bc6d9
commit 29c8a27e01

View File

@ -104,7 +104,7 @@ jobs:
- name: Run ESLint (admin-frontend)
run: |
cd admin-frontend
npm run lint || true
npm run lint
continue-on-error: true
- name: Install viewer-frontend dependencies
@ -121,7 +121,7 @@ jobs:
- name: Type check (viewer-frontend)
run: |
cd viewer-frontend
npm run type-check || true
npm run type-check
continue-on-error: true
python-lint:
@ -146,12 +146,12 @@ jobs:
- name: Check Python syntax
run: |
find backend -name "*.py" -exec python -m py_compile {} \; || true
find backend -name "*.py" -exec python -m py_compile {} \;
continue-on-error: true
- name: Run flake8
run: |
flake8 backend --max-line-length=100 --ignore=E501,W503 || true
flake8 backend --max-line-length=100 --ignore=E501,W503
continue-on-error: true
test-backend: