Skip to content

fix(deploy): ignore line-ending-only changes #66

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
jackseg80 merged 1 commit into from
Sep 26, 2026
Merged

fix(deploy): ignore line-ending-only changes #66

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
2 changes: 2 additions & 0 deletions deploy.sh
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,8 @@ fi
echo -e "${YELLOW} Step 1/5: Checking for local changes...${NC}"
if git diff --quiet && git diff --cached --quiet; then
echo -e "${GREEN}[OK] No local changes${NC}"
elif git diff --ignore-space-at-eol --quiet && git diff --cached --ignore-space-at-eol --quiet; then
echo -e "${GREEN}[OK] Only line-ending differences detected; no reset needed${NC}"
else
echo -e "${YELLOW}[Warning] Local changes detected:${NC}"
git status --short
Expand Down
4 changes: 2 additions & 2 deletions docs/DOCKER_DEPLOYMENT.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -167,7 +167,7 @@ Le script `deploy.sh` est la méthode **recommandée** pour tous les déploiemen

### Processus du script
Le script exécute les étapes suivantes :
1. **Vérification des changements locaux** : Si des modifications existent sur le serveur, il propose de les sauvegarder dans un patch avant de les écraser.
1. **Vérification des changements locaux** : Les différences limitées aux fins de ligne sont ignorées. Pour les autres modifications suivies par Git, le script propose une sauvegarde dans un patch puis leur remplacement après confirmation. Les fichiers non suivis ne déclenchent pas cette vérification et ne sont pas supprimés par le reset.
2. **Pull depuis GitHub** : Récupère la dernière version du code.
3. **Vérification du cache de prix** : Vous alerte si le cache semble incomplet.
4. **Reconstruction & Redémarrage Docker** : Reconstruit l'image de l'API et relance la stack.
Expand Down Expand Up @@ -309,4 +309,4 @@ echo "[OK] Backup créé: $BACKUP_DIR.tar.gz"
```

### Restauration
La restauration implique de stopper les services, de remplacer les données par celles du backup, et de redémarrer. Assurez-vous de bien comprendre le processus avant de le tenter.
La restauration implique de stopper les services, de remplacer les données par celles du backup, et de redémarrer. Assurez-vous de bien comprendre le processus avant de le tenter.
2 changes: 1 addition & 1 deletion docs/audit/STOCK_MARKET_2026-09-26.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -139,4 +139,4 @@ La preuve de pertinence financière reste ouverte : conserver des signaux et fon

Les listes de candidats restent un univers limité ; disponibilité chez Saxo, liquidité, frais, contraintes fiscales, transparisation des fonds et validation financière hors échantillon ne sont pas attestés par ce complément. Le portefeuille de production actuel n'a pas été rapproché d'un export récent.

État de ce complément : modifications locales testées et documentées, pas encore committées/poussées/fusionnées ni déployées sur Robot2. Un simple `deploy.sh` ne peut pas récupérer ce travail tant qu'il n'est pas publié dans `main`.
Mise à jour du 2026-09-26 : le complément a été fusionné dans `main` via la PR #65 (commit `631a2bdf`) et son déploiement sur Robot2 a été confirmé par l’utilisateur. La pertinence financière hors échantillon, la comparaison au portefeuille réel et la validation Saxo restent à établir avec des données datées et des résultats observés ; le déploiement ne constitue pas cette preuve.
Loading