Skip to content

Conversation

@bastiendmt
Copy link
Contributor

Re @tdd, bizarrement je n'avais toutes les suggestions dans l'onglet files 🫤 j'ai appliqué les corrections à la main mais en plusieurs commit du coup.
J'ai fait les correctifs en deux fois entre 12h et 14h mais tu as merge entre temp (je n'avais pas fait attention au status). Voici la suite de ma précédente PR #455

Quelle est la bonne pratique normalement ? demander une nouvelle review ?

@github-actions
Copy link

Size changes

Details

📦 Next.js Bundle Analysis for react-dev

This analysis was generated by the Next.js Bundle Analysis action. 🤖

This PR introduced no changes to the JavaScript bundle! 🙌

@tdd tdd merged commit 4810295 into reactjs:main Apr 27, 2023
@tdd tdd self-assigned this Apr 27, 2023
@tdd tdd added Reviewed scope:required-pages Part of the required translations before we can ship translator:beginner The author of this translation has little to no prior doc translation experience labels Apr 27, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Reviewed scope:required-pages Part of the required translations before we can ship translator:beginner The author of this translation has little to no prior doc translation experience

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants