gly-engine / gly-engine/gly-type
r-lyeh suggestions
- Dominant language
- C
- Stars
- 11
- Forks
- 2
- PR merge metrics
- No merged PRs in 30d
Description
Hey thanks. Couple of suggestions for your project:
- The header is probably not usable from 2 different C sources in its current form. Either a static inline or a #define SOMETHING_IMPLEMENTATION is missing (the latter is preferred in this kind of libraries)
- Consider having screenshots in your readme, so others can see what the font face is about.
_Originalmente postado por @r-lyeh em https://github.com/r-lyeh/single_file_libs/pull/294#issuecomment-2664071965_
Contributor guide
No contributing guide indexed for this repository
Research direction
Start by inspecting the project header and how it is included from multiple C sources, then review the README's current presentation of the font face. The work is done when the header can be used from two C sources without conflicts and the README includes screenshots showing the font renderer.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- c
- Domain
- computer-graphics
- Issue type
- Feature
- Difficulty
- 3/5
- Estimated time
- 1-2 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100