docs: Harmonize lis2mdl README — remove emojis.#208
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the LIS2MDL driver README to match the repo’s documentation style by removing emojis from section headings (per issues #195 and #194).
Changes:
- Removed emojis from all
##section titles inlib/lis2mdl/README.mdwithout altering the surrounding content.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
nedseb
left a comment
nedseb
left a comment
There was a problem hiding this comment.
Salut Charly,
PR simple et bien ciblée. Le diff fait exactement ce qui est demandé : retirer les emojis des titres de section. Commit message correct, bon fichier modifié, pas de changement de contenu.
Un seul point à corriger :
🟡 Les ✅ dans la section Features sont encore là
L'issue demande de retirer tous les emojis. Les titres sont corrigés, mais la section Features contient encore 9 lignes avec des ✅ :
* ✅ Full **I²C driver** for LIS2MDL
* ✅ Supports **10 / 20 / 50 / 100 Hz** output data rates
* ✅ Temperature-compensated and low-power modes
...Ces ✅ sont des emojis (U+2705). L'issue dit "Remove all emojis" et la checklist dit "All emojis removed from section titles" — on peut interpréter "section titles" comme les ## uniquement, mais les ✅ dans les listes sont aussi hors norme par rapport aux autres README du projet (aucun autre driver n'utilise de ✅ dans ses features). Retire-les pour être cohérent.
En dehors de ça, c'est bon. Corrige les ✅ et ça sera prêt à merger.
|
Deleted all Emoji's Should be ready to merge |
|
🎉 This PR is included in version 0.0.2 🎉 The release is available on GitHub release Your semantic-release bot 📦🚀 |
Closes #195
Parents issue: #194
Remove all emojis from section titles in lib/lis2mdl/README.md.