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

Logements should keep the importation order #1604

Draft
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

syldb
Copy link
Contributor

@syldb syldb commented Oct 7, 2024

Mes tests en local semblent montrer que le .all() conserve l'ordre d'import qu'on avait sur le docx...
Je me demande à quel point c'est fiable, j'aimerais tester en staging.

(Sinon il faudra un champ pour stocker l'ordre)

@syldb syldb requested a review from a team as a code owner October 7, 2024 09:17
@syldb syldb requested review from kolok and etchegom and removed request for a team October 7, 2024 09:17
@syldb syldb marked this pull request as draft October 7, 2024 09:18
@syldb syldb force-pushed the tableau-logement-ordre-import branch from 5feac7b to 3ef8bc8 Compare October 7, 2024 09:31
Copy link

sonarcloud bot commented Oct 7, 2024

Copy link

github-actions bot commented Oct 7, 2024

☂️ Python Coverage

current status: ✅

Overall Coverage

Lines Covered Coverage Threshold Status
8284 6888 83% 80% 🟢

New Files

No new covered files...

Modified Files

File Coverage Status
conventions/services/convention_generator.py 83% 🟢
TOTAL 83% 🟢

updated for commit: 3ef8bc8 by action🐍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant