Skip to content

Optimize build and test workflow#540

Open
DaijobuDes wants to merge 3 commits intobettergovph:mainfrom
DaijobuDes:optimize-build-and-test-ci
Open

Optimize build and test workflow#540
DaijobuDes wants to merge 3 commits intobettergovph:mainfrom
DaijobuDes:optimize-build-and-test-ci

Conversation

@DaijobuDes
Copy link
Member

Summary

  • Temporarily commented out build_image since it does not push anything and wastes time.
  • Add npm run build under lint_checks step to test if the application builds correctly and skip e2e_tests if it fails to save time.

Sample workflow ran: https://github.com/DaijobuDes/bettergov/actions/runs/19323461242

- Temporarily commented out build_image since it has no use for now
- Add npm run build to test if app builds correctly and skip e2e_tests
if it fails
@DaijobuDes DaijobuDes self-assigned this Nov 13, 2025
@DaijobuDes DaijobuDes added low priority This project can take it easy, no to long deadline. ci labels Nov 13, 2025
@j4ckofalltrades
Copy link
Contributor

Suggested an alternative way of disabling the build_image job but either way is fine.

LGTM.

@j4ckofalltrades
Copy link
Contributor

j4ckofalltrades commented Nov 17, 2025

@DaijobuDes This change needs to be merged in manually (by someone that has write access / someone that can override repo rules). GitHub Actions has stalled and is waiting for the Build image (which has been disabled) and Lint (renamed to Lint and Build) to run.

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

Labels

ci low priority This project can take it easy, no to long deadline.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants