Skip to content

ci(webapp): add lint and build steps to webapp CI workflow#182

Open
alikhere wants to merge 1 commit intoc2siorg:mainfrom
alikhere:ci/add-lint-build-workflow
Open

ci(webapp): add lint and build steps to webapp CI workflow#182
alikhere wants to merge 1 commit intoc2siorg:mainfrom
alikhere:ci/add-lint-build-workflow

Conversation

@alikhere
Copy link
Copy Markdown

@alikhere alikhere commented Mar 24, 2026

Summary

The webapp CI workflow only ran tests — a PR that broke the build or introduced lint errors would ship green. Added two steps
after the existing test step:

  • Lint runs npm run lint with continue-on-error: true because there are pre-existing lint errors in the repo unrelated to any individual PR. This way lint output is visible in CI without blocking merges until those are cleaned up separately.
  • Build runs npm run build and must pass. A broken build now blocks merges.

How to test

  • Open a PR against main and check the Run NPM Tests on Pull Request workflow run
  • Lint step should appear and report results (may show warnings/errors from pre-existing issues) without failing the job
  • Build step should appear and pass; introduce a syntax error to verify it fails and blocks the merge

FIxes #181

  steps to webapp CI workflow
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

CI: add lint and build steps to frontend workflow

1 participant