Skip to content

team: bilingual EN/FR report#16

Merged
pierrick-fonquerne merged 5 commits into
mainfrom
feature/team-bilingual-report
Jun 10, 2026
Merged

team: bilingual EN/FR report#16
pierrick-fonquerne merged 5 commits into
mainfrom
feature/team-bilingual-report

Conversation

@pierrick-fonquerne

Copy link
Copy Markdown
Contributor

Scope

  • The synthesis and verification steps now produce every human-readable string in English and French (no extra API calls). French fields are optional with serde defaults, so a model that returns English only still deserializes and renders.
  • The PR comment shows a shared header (verdict and counts), then an English block and a French block (open by default) as collapsible sections.
  • Confirmed minor findings are collapsed under a nested section to keep the comment short. Inline comments show French first, English second.

Acceptance criteria

  • The comment contains both an English and a French collapsible block (French open by default).
  • Missing French fields fall back to English (tolerant deserialization).
  • Minor confirmed findings are collapsed; criticals stay visible.
  • fmt + clippy (all/pedantic, -D warnings) + tests are green (36 tests).

@pierrick-fonquerne pierrick-fonquerne merged commit bc8fba7 into main Jun 10, 2026
3 checks passed
@pierrick-fonquerne pierrick-fonquerne deleted the feature/team-bilingual-report branch June 10, 2026 13:36
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