Conversation
![sourcery-ai[bot]](https://avatars.githubusercontent.com/in/48477?s=60&v=4){kind=small}
There was a problem hiding this comment.
Sorry @gridhead, your pull request is larger than the review limit of 150000 diff characters
![gemini-code-assist[bot]](https://avatars.githubusercontent.com/in/956858?s=60&v=4){kind=small}
There was a problem hiding this comment.
Code Review
This pull request adds a new character, Lohen, along with their asset definitions, character data, and unit tests. The feedback highlights that the newly added entries for Lohen in the resource files (back.qrc, face.qrc, name.qrc, wish.qrc) and the CharName enum are out of alphabetical order and should be placed after 'lisa' to maintain consistency.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
| <file>imgs/char/back/lauma.webp</file> | ||
| <file>imgs/char/back/layla.webp</file> | ||
| <file>imgs/char/back/linnea.webp</file> | ||
| <file>imgs/char/back/lohen.webp</file> |
There was a problem hiding this comment.
The entry for lohen.webp is out of alphabetical order. It should be placed after lisa.webp (line 67) to maintain proper alphabetical sorting.
| <file>imgs/char/face/lauma.webp</file> | ||
| <file>imgs/char/face/layla.webp</file> | ||
| <file>imgs/char/face/linnea.webp</file> | ||
| <file>imgs/char/face/lohen.webp</file> |
There was a problem hiding this comment.
The entry for lohen.webp is out of alphabetical order. It should be placed after lisa.webp (line 67) to maintain proper alphabetical sorting.
| <file>imgs/char/name/lauma.webp</file> | ||
| <file>imgs/char/name/layla.webp</file> | ||
| <file>imgs/char/name/linnea.webp</file> | ||
| <file>imgs/char/name/lohen.webp</file> |
There was a problem hiding this comment.
The entry for lohen.webp is out of alphabetical order. It should be placed after lisa.webp (line 67) to maintain proper alphabetical sorting.
| <file>imgs/char/wish/lauma.webp</file> | ||
| <file>imgs/char/wish/layla.webp</file> | ||
| <file>imgs/char/wish/linnea.webp</file> | ||
| <file>imgs/char/wish/lohen.webp</file> |
There was a problem hiding this comment.
The entry for lohen.webp is out of alphabetical order. It should be placed after lisa.webp (line 67) to maintain proper alphabetical sorting.
| lauma = "Lauma" | ||
| layla = "Layla" | ||
| linnea = "Linnea" | ||
| lohen = "Lohen" |
There was a problem hiding this comment.
The enum member lohen is out of alphabetical order. It should be placed after lisa (line 83) to maintain proper alphabetical sorting.
gridhead
force-pushed
the
wild
branch
2 times, most recently
from
508d964 to
bbdb09d
Compare
Co-authored-by: Akashdeep Dhar <akashdeep.dhar@gmail.com> Co-authored-by: Claude Code <noreply@anthropic.com>
1 participant
Introduce the recently added character
Lohento the rosterFixes #537