-
Notifications
You must be signed in to change notification settings - Fork 72
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Photo of the font being used in a tattoo #134
Conversation
Reviewer's Guide by SourceryThis PR updates the README.md file to rename the "Screenshots" section to "Type Specimen" and adds a new image showing the font being used in a tattoo. No diagrams generated as the changes look simple and do not need a visual representation. File-Level Changes
Tips and commandsInteracting with Sourcery
Customizing Your ExperienceAccess your dashboard to:
Getting Help
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hey @felipesanches - I've reviewed your changes - here's some feedback:
Overall Comments:
- While it's great to see enthusiasm for the font, the README should focus on technical documentation. Consider removing the tattoo image and keeping the original 'Screenshots' heading since that better describes the section's content.
Here's what I looked at during the review
- 🟢 General issues: all looks good
- 🟢 Security: all looks good
- 🟢 Testing: all looks good
- 🟢 Complexity: all looks good
- 🟢 Documentation: all looks good
Help me be more useful! Please click 👍 or 👎 on each comment and I'll use the feedback to improve your reviews.
Can you make this, but for the develop branch? |
Summary by Sourcery
Documentation: