ci: consolidate and improve workflows #113

Merged

addison merged 2 commits from exe-dev-bot/karriba:ci/improved-workflows into main

2026-01-24 06:35:18 -05:00

exe-dev-bot commented

2026-01-24 06:27:27 -05:00

Collaborator

Learned from Daily-Manna workflows:

Merge build + comment into single PR Preview workflow (no more workflow_run trigger)
Add preview version stamping with PR number and short SHA
Add flutter_test.yml for running tests on PRs
Remove separate comment_artifact_url_on_pr.yml (no longer needed)
Use Forgejo-compatible API calls for PR comments
Update action versions and add retention-days for artifacts

Learned from [Daily-Manna workflows](https://github.com/el-apps/Daily-Manna/tree/main/.github/workflows): - Merge build + comment into single PR Preview workflow (no more workflow_run trigger) - Add preview version stamping with PR number and short SHA - Add `flutter_test.yml` for running tests on PRs - Remove separate `comment_artifact_url_on_pr.yml` (no longer needed) - Use Forgejo-compatible API calls for PR comments - Update action versions and add retention-days for artifacts

exe-dev-bot added 1 commit

2026-01-24 06:27:27 -05:00

ci: consolidate and improve workflows 96853c8075

- Merge build + comment into single PR Preview workflow
- Add preview version stamping with PR number and short SHA
- Add flutter_test.yml for running tests on PRs
- Remove separate comment_artifact_url_on_pr.yml (no longer needed)
- Use Forgejo-compatible API calls for PR comments
- Update action versions and add retention-days for artifacts

Co-authored-by: Shelley <shelley@exe.dev>

exe-dev-bot added 1 commit

2026-01-24 06:30:59 -05:00

fix(ci): address review feedback 75a25f5620

- Fix newline escaping using printf
- Remove jq dependency, use grep instead
- Add error handling for curl failures
- Enable Flutter caching for faster builds

Co-authored-by: Shelley <shelley@exe.dev>