:code: |
Meaning | |
|---|---|---|
| 👍💯😍 | :+1: :100:, :heart_eyes: |
I like this... ...and I want the author to know it! This is a way to highlight positive parts of a code review. |
| 🔧 | :wrench: |
I think this needs to be changed This is a question, a concern, or suggested change/refactor that I feel is worth addressing before approving. |
| ❓ | :question: |
I have a question This should be a fully formed question with sufficient information and context that requires a response. |
| 🤔💭 | :thinking: :thought_balloon: |
Let me think out loud here for a minute I might express concern, suggest an alternative solution, or walk through the code in my own words to make sure I understand. |
| 🌱 | :seedling: |
Planting a seed for future An observation or suggestion that is not a change request, but may have larger implications. Generally something to keep in mind for the future. |
| 📝 | :memo: |
Here's a note This is an explanatory note, fun fact, or relevant commentary that does not require any action. |
| ♻️ | :recycle: |
Suggestion for an optional refactoring Should include enough context to be actionable and not be considered a nitpick. |
| 🏕 | :camping: |
Opportunity Here is an opportunity, not directly related to your changes, for us to leave the campground [code] cleaner than we found it. |
| 📌 | :pushpin: |
Don't forget This is a concern that is out of scope and should be staged appropriately for follow up. |
| 💡 | :bulb: |
I didn't know that! This is something I'm learning thanks to the author and the review. |
:warning: |
I noticed something that might be an error I trust the author to dig deeper on this or dismiss it if non-relevent. |
|
| 🍖 | :meat_on_bone: |
Food for thoughts Do what you want with it but it's totally okay to dismiss it. It could be about how I would have done this or that, external references, and so on. |
Taken from Nicolas Zermati's legend: gist
Also duplicates the Code Review Emoji guide from Erik Aybar