Fix capitalization of 'Tabby' in README#129
Merged
Merged
Conversation
| ## Creators | ||
|
|
||
| Tabby is built by [@FuJacob](https://github.com/FuJacob) and [@jam-cai](https://github.com/jam-cai). | ||
| tabby is built by [@FuJacob](https://github.com/FuJacob) and [@jam-cai](https://github.com/jam-cai). |
Contributor
There was a problem hiding this comment.
Starting a sentence with a lowercase brand name is grammatically awkward. Even though the project is stylized as
tabby, the sentence tabby is built by... begins a new sentence with a lowercase letter, which many style guides flag. Consider restructuring to avoid opening a sentence with the brand name.
Suggested change
| tabby is built by [@FuJacob](https://github.com/FuJacob) and [@jam-cai](https://github.com/jam-cai). | |
| **tabby** is built by [@FuJacob](https://github.com/FuJacob) and [@jam-cai](https://github.com/jam-cai). |
Note: If this suggestion doesn't match your team's coding style, reply to this and let me know. I'll remember it for next time!
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Validation
Linked issues
Risk / rollout notes
Greptile Summary
This PR updates the "Creators" section of
README.mdto lowercase the product name fromTabbytotabby, aligning with the project's established lowercase branding convention used throughoutAGENTS.md, the README title (# tabby), and the rest of the document.tabby is built by..., which starts a sentence with a lowercase letter — grammatically awkward regardless of brand style. Restructuring the sentence (e.g. leading with "Built by" or bolding the brand name) would resolve the tension between brand casing and sentence-start conventions.Confidence Score: 4/5
This is a documentation-only change with no risk to functionality.
The change correctly aligns brand casing with the rest of the document, but the resulting sentence now opens with a lowercase word, which is grammatically unconventional and worth a small tweak before merging.
README.md — the Creators sentence now starts with a lowercase brand name.
Important Files Changed
Flowchart
%%{init: {'theme': 'neutral'}}%% flowchart TD A[README.md Creators section] --> B{Brand name at sentence start} B -->|Before PR| C["'Tabby is built by...' — capitalized, grammatically correct but inconsistent with brand"] B -->|After PR| D["'tabby is built by...' — matches brand style but starts sentence with lowercase"] D --> E{Possible improvements} E --> F["'**tabby** is built by...' — bold keeps brand identity, signals sentence start"] E --> G["'Built by @FuJacob and @jam-cai — restructure to avoid brand at sentence start"]Reviews (1): Last reviewed commit: "Fix capitalization of 'Tabby' in README" | Re-trigger Greptile