-
Notifications
You must be signed in to change notification settings - Fork 77
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
Update documentation to match changes of IGN #3840
Conversation
Passing run #7736 ↗︎
Details:
Review all test suite changes for PR #3840 ↗︎ |
09f235e
to
f9c3f29
Compare
@submarcos je veux bien un coup de main sur cette PR ? Pas certain de bien voir pourquoi il y a des tests qui sont KO. Si jamais tu as quelques instants (toi ou quelqu'un d'autre) je veux bien que vous repreniez ce travail pendant mon absence. Merci 🙏 |
@babastienne c'est surement le changement d'ordre des fonds de plan le soucis. Il vaut mieux separer les modifs, une PR pour la doc, une pour les trads, une pour l'ordre par defaut des fonds de plan |
71d0a23
to
237d800
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #3840 +/- ##
=======================================
Coverage 98.34% 98.34%
=======================================
Files 292 292
Lines 22048 22048
=======================================
Hits 21684 21684
Misses 364 364 ☔ View full report in Codecov by Sentry. |
Description
Related Issue
Checklist
I have added tests that prove my fix is effective or that my feature works.New and existing unit tests pass locally with my changesMy commits are all using prefix convention (emoji + tag name) and references associated issuesThe title of my PR mentionned the issue associated