Skip to content

ci: check web JavaScript syntax - #176

Merged
LeyckerS merged 1 commit into
LeyckerS:mainfrom
tunglambk:ci/web-syntax-check
Aug 29, 2026
Merged

ci: check web JavaScript syntax#176
LeyckerS merged 1 commit into
LeyckerS:mainfrom
tunglambk:ci/web-syntax-check

Conversation

@tunglambk

Copy link
Copy Markdown
Contributor

Closes #174

Description

Adds a dedicated GitHub Actions workflow for changes under web/**. It runs node --check web/app.js, so JavaScript syntax errors block the pull request before they reach the GUI. The workflow also watches its own file so future gate changes are checked.

Local validation: node --check web/app.js, YAML parsing, and git diff --check.

Type of change

  • Bug fix (non-breaking change that fixes an issue)
  • New feature (non-breaking change that adds functionality)
  • Breaking change (fix or feature that would change existing behavior)
  • Documentation update
  • Refactor / code cleanup
  • Other: CI

Checklist

  • I have tested my changes locally
  • Shared extraction and download logic is unaffected
  • I have kept the single-file architecture (no package split)
  • I have not added new dependencies

Screenshots / logs (if applicable)

Not applicable.

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.

PRs that touch only web/ run no CI at all — not even a syntax check

2 participants