AOSSIE-Org / AOSSIE-Org/PictoPy

Improve PR CI workflow: readability, caching, and validation

オープン
#818 コメント 4 件 リアクション 1 件 担当者 1 名 @akshajtiwari が担当を希望しています GitHub で見る
CI/CD enhancement
主要言語
Python
スター
283
フォーク
679
平均マージ
7日 2時間
マージ済み PR(30日)
3

説明

## Abstract

The current PR CI workflow (`pr-check-tests.yml`) is functionally correct but has several **readability, maintainability, and minor performance concerns** identified during review. Addressing these issues will improve long-term maintainability, reviewer clarity, and CI efficiency **without changing the core behavior** of the pipeline.

This issue proposes a **targeted cleanup and enhancement** of the existing workflow while **preserving the original directory structure and intent**.

---

## Proposed Improvements

- Standardize section comment indentation and placement for better readability
- Remove redundant installation of `ruff` and `black` (already managed by `pre-commit`)
- Use `working-directory` consistently to improve clarity and reduce shell state reliance
- Simplify Python setup by removing the single-version matrix configuration
- Clarify PyInstaller’s role as a **build-validation step**, not an artifact delivery mechanism
- Add basic artifact existence checks after PyInstaller builds
- Extend pip caching to cover both the backend and sync microservice dependencies
- Add backend coverage reporting using `pytest-cov`

These changes **do not alter the logical flow** of the workflow and keep directory usage **exactly as in the original design**.

---

## Expected Outcome

- Cleaner, more readable CI workflow
- Faster CI execution through improved caching
- Clearer intent behind build-related steps
- Better visibility into backend test health via coverage reporting
- Easier future extensions (e.g., additional Python versions, artifact uploads)

---

## Scope

- **File affected:** `.github/workflows/pr-check-tests.yml`
- **No changes** to application logic or test behavior
- **CI-only** improvement

--
## Pull Requests

I have made some changes and put up a Pull request for the same in #672 , commit akshajtiwari@d631a0e

コントリビューションガイド

コントリビューションガイドを開く

評価

この issue はまだ評価されていません。

新しい issue をメールで受け取る

初心者向けの GitHub issue を短くまとめたダイジェスト。