Skip to content
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.

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

[DO NOT MERGE] Hotfix ajout du financement SALLS_R #1644

Closed
wants to merge 2 commits into from
Closed

Conversation

syldb
Copy link
Contributor

@syldb syldb commented Nov 7, 2024

Description succincte du problème résolu

Carte Airtable / Lien Mattermost : Titre

Description plus détaillée de l'intention, l'approche ou de l'implémentation (ce qui n’est pas visible directement en lisant le code)

Type de changement :

  • Bug fix
  • Nouvelle fonctionnalité
  • Mise à jour de données / DAG
  • Les changements nécessitent une mise à jour de documentation
  • Refactoring de code (explication à retrouver dans la description)

Auto-review

Les trucs à faire avant de demander une review :

  • J'ai bien relu mon code
  • La CI passe bien
  • En cas d'ajout de variable d'environnement, j'ai bien mis à jour le .env.template
  • J'ai ajouté des tests qui couvrent le nouveau code

Comment tester

En local / staging :

@syldb syldb changed the title Ajout du financement SALLS_R (#1643) [DO NOT MERGE] Ajout du financement SALLS_R Nov 7, 2024
@syldb syldb changed the title [DO NOT MERGE] Ajout du financement SALLS_R [DO NOT MERGE] Hotfix ajout du financement SALLS_R Nov 7, 2024
Copy link

github-actions bot commented Nov 7, 2024

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
8307 6906 83% 80% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
programmes/models/choices.py 96% 🟢
TOTAL 96% 🟢

updated for commit: 0fad68f by action🐍

Copy link

sonarqubecloud bot commented Nov 7, 2024

@syldb syldb closed this Nov 7, 2024
@syldb syldb added the technical label Nov 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants