Improve CI Pipelines: Linting, Formatting, Pre-commits, and Testing
@sahilds1 ci sta già lavorando.
Dal 8/7/2026.
Valutazione
Questa issue non è ancora stata valutata.
Descrizione
The current CI workflows in .github/workflows/frontend-ci.yml and .github/workflows/python-app.yml are missing several critical quality checks.
Proposed Improvements:
1. Frontend (frontend-ci.yml)
- Strict Checks: Remove
continue-on-error: truefromLintandBuildsteps so that PRs cannot be merged with failures. - Formatting: Add a step to verify formatting using Prettier (e.g.,
npx prettier --check .). - Testing: Add a step to run tests if any exist (e.g.,
npm test). - Dependency Cache: Ensure Node.js cache is working correctly (already present but should be verified).
2. Python Backend (python-app.yml)
- Formatting: Add Ruff formatting check (
ruff format --check). - Testing: Add a step to run Django tests (
python manage.py test). - Branch Triggers: Update branch triggers to include
developor the main branch (currently points tolistOfMed). - Linting: Improve Ruff linting configuration or ensure it's running with the desired rules.
3. General Enhancements
- Pre-commit Hooks: Integrate
pre-commitinto CI to ensure all hooks pass before merging. - Consistency: Align branch triggers across all CI workflows.
Why is this needed?
To ensure code quality, consistency, and prevent regressions from being merged into the main branches.
- Lingua principale
- TypeScript
- Stelle
- 21
- Fork
- 18
- Metriche di merge delle PR
- Nessuna PR unita negli ultimi 30g
Preparare l'ambiente
- Include un Dockerfile o un file Docker Compose
- Ha un modello di pull request
- Nessuna guida per i contributori
Come iniziare
- Leggi tutta la issue e poi la guida ai contributi del progetto.
- Commenta sulla issue per dire che te ne occupi tu — evita che due persone facciano lo stesso lavoro.
- Fai un fork del repository e lavora su un branch.
- Apri una pull request che faccia riferimento al numero della issue.
Altre issue di CodeForPhilly/balancer-main
-
Rename the "Click to Enter New Patient" Button to say "Enter New Patient"Forse di nuovo libera Una pull request per questa issue è stata chiusa senza essere unita. ApertaBug
Difficoltà 1/5 1-3 ore Idoneità per principianti 72/100
CodeForPhilly/balancer-main#509 ·
-
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 76/100
CodeForPhilly/balancer-main#503 ·
-
Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
CodeForPhilly/balancer-main#502 · 1 commento ·
-
Bug
Difficoltà 2/5 1-3 ore Idoneità per principianti 65/100
CodeForPhilly/balancer-main#488 ·
-
Documentation
Difficoltà 2/5 1-3 ore Idoneità per principianti 62/100
CodeForPhilly/balancer-main#478 ·
Tutte le issue di CodeForPhilly/balancer-main
Issue simili
-
Remove the landing pageApertaby: ai-assisted frontend good-for: new-member spike
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Northeastern-Electric-Racing/Argos#847 ·
I maintainer di solito rispondono entro 4 giorni
-
Difficoltà 1/5 1-3 ore Idoneità per principianti 84/100
SignalK/freeboard-sk#990 ·
I maintainer di solito rispondono entro 1 giorno
-
[missing-inheritance] audit review (1 preset)Forse già presa @github-actions l’ha presa oggi. Aperta
Difficoltà 1/5 Meno di un'ora Idoneità per principianti 82/100
osmberlin/tagging-schema-browser#363 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno
-
enhancement
Difficoltà 2/5 1-3 ore Idoneità per principianti 68/100
Albert-Weasker/niubigeo#205 ·
I maintainer di solito rispondono entro 1 giorno
-
area/frontend area/v2 kind/bug priority/needs-triage
Difficoltà 2/5 1-3 ore Idoneità per principianti 70/100
kubeflow/notebooks#1498 · 1 commento ·
I maintainer di solito rispondono entro 1 giorno