Skip to content

Add GitHub Actions CI with backend tests and frontend lint/build#406

Merged
mahmoud merged 1 commit intomasterfrom
add-github-ci
Mar 10, 2026
Merged

Add GitHub Actions CI with backend tests and frontend lint/build#406
mahmoud merged 1 commit intomasterfrom
add-github-ci

Conversation

@mahmoud
Copy link
Member

@mahmoud mahmoud commented Mar 10, 2026

  • Add orchestrating workflow (pr.yml) with path-based change detection using dorny/paths-filter, concurrency cancellation, and ci-complete sentinel job for branch protection

  • Add backend test workflow: builds Docker image, runs pytest on test_web_basic.py, verifies module imports

  • Add frontend lint workflow: prettier --check, eslint, vite build

  • Add pytest to dockerfile

  • Add lint:check and format:check scripts to frontend/package.json

  • Fix 6 pre-existing ESLint errors:

    • Remove unused defineComponent import (ViewCampaign.vue)
    • Remove unused downloadReviews function (RoundInfo.vue)
    • Remove unused props assignment (RoundView.vue)
    • Add eslint-disable for multi-word component name (Vote.vue)
    • Remove unused getCommonsImageUrl imports (VoteEdit.vue, VoteRanking.vue)
  • Remove pdb.set_trace() calls from test_web_basic.py (would hang CI)

  • Run prettier --write across frontend/src/ to establish baseline

  • Delete dead .travis.yml (Python 2.7, Travis CI abandoned)

- Add orchestrating workflow (pr.yml) with path-based change detection
  using dorny/paths-filter, concurrency cancellation, and ci-complete
  sentinel job for branch protection

- Add backend test workflow: builds Docker image, runs pytest on
  test_web_basic.py, verifies module imports

- Add frontend lint workflow: prettier --check, eslint, vite build

- Add pytest to dockerfile

- Add lint:check and format:check scripts to frontend/package.json

- Fix 6 pre-existing ESLint errors:
  - Remove unused defineComponent import (ViewCampaign.vue)
  - Remove unused downloadReviews function (RoundInfo.vue)
  - Remove unused props assignment (RoundView.vue)
  - Add eslint-disable for multi-word component name (Vote.vue)
  - Remove unused getCommonsImageUrl imports (VoteEdit.vue, VoteRanking.vue)

- Remove pdb.set_trace() calls from test_web_basic.py (would hang CI)

- Run prettier --write across frontend/src/ to establish baseline

- Delete dead .travis.yml (Python 2.7, Travis CI abandoned)
@mahmoud mahmoud merged commit 8f06934 into master Mar 10, 2026
4 checks passed
ItsAbhinavM pushed a commit to ItsAbhinavM/montage that referenced this pull request Mar 11, 2026
…note#406)

- Add orchestrating workflow (pr.yml) with path-based change detection
  using dorny/paths-filter, concurrency cancellation, and ci-complete
  sentinel job for branch protection

- Add backend test workflow: builds Docker image, runs pytest on
  test_web_basic.py, verifies module imports

- Add frontend lint workflow: prettier --check, eslint, vite build

- Add pytest to dockerfile

- Add lint:check and format:check scripts to frontend/package.json

- Fix 6 pre-existing ESLint errors:
  - Remove unused defineComponent import (ViewCampaign.vue)
  - Remove unused downloadReviews function (RoundInfo.vue)
  - Remove unused props assignment (RoundView.vue)
  - Add eslint-disable for multi-word component name (Vote.vue)
  - Remove unused getCommonsImageUrl imports (VoteEdit.vue, VoteRanking.vue)

- Remove pdb.set_trace() calls from test_web_basic.py (would hang CI)

- Run prettier --write across frontend/src/ to establish baseline

- Delete dead .travis.yml (Python 2.7, Travis CI abandoned)
ademolaomosanya pushed a commit to ademolaomosanya/montage that referenced this pull request Mar 11, 2026
…note#406)

- Add orchestrating workflow (pr.yml) with path-based change detection
  using dorny/paths-filter, concurrency cancellation, and ci-complete
  sentinel job for branch protection

- Add backend test workflow: builds Docker image, runs pytest on
  test_web_basic.py, verifies module imports

- Add frontend lint workflow: prettier --check, eslint, vite build

- Add pytest to dockerfile

- Add lint:check and format:check scripts to frontend/package.json

- Fix 6 pre-existing ESLint errors:
  - Remove unused defineComponent import (ViewCampaign.vue)
  - Remove unused downloadReviews function (RoundInfo.vue)
  - Remove unused props assignment (RoundView.vue)
  - Add eslint-disable for multi-word component name (Vote.vue)
  - Remove unused getCommonsImageUrl imports (VoteEdit.vue, VoteRanking.vue)

- Remove pdb.set_trace() calls from test_web_basic.py (would hang CI)

- Run prettier --write across frontend/src/ to establish baseline

- Delete dead .travis.yml (Python 2.7, Travis CI abandoned)
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.

1 participant