Add sales entry to showcases section in repo's main readme (#9837)#9838
Conversation
WalkthroughThis pull request updates the OSS Showcases section in the README.md file. It adds a new entry for a Sales application, linked to https://sales.bitplatform.dev, and adjusts the numbering of existing entries for AdminPanel and Todo. The Sales entry is inserted as number 9 while the remaining entries are renumbered to maintain the overall structure. Changes
Assessment against linked issues
Poem
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media? 🪧 TipsChatThere are 3 ways to chat with CodeRabbit:
Note: Be mindful of the bot's finite context window. It's strongly recommended to break down tasks such as reading entire modules into smaller chunks. For a focused discussion, use review comments to chat about specific files and their changes, instead of using the PR comments. CodeRabbit Commands (Invoked using PR comments)
Other keywords and placeholders
CodeRabbit Configuration File (
|
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 0
🧹 Nitpick comments (2)
README.md (2)
45-46: Table Row Column Count Inconsistency in OSS ShowcasesThe "Sales" entry added on line 45—and the adjacent "bitplatform" row on line 46—provide only two columns, whereas the table header (lines 40–41) specifies six columns. This mismatch may lead to rendering issues in Markdown. Please add placeholder cells (e.g., "-" or "N/A") for the missing columns if no content is available.
🧰 Tools
🪛 markdownlint-cli2 (0.17.2)
45-45: Table column count
Expected: 6; Actual: 2; Too few cells, row will be missing data(MD056, table-column-count)
46-46: Table column count
Expected: 6; Actual: 2; Too few cells, row will be missing data(MD056, table-column-count)
54-55: Terminology Consistency for "Todo" BrandingStatic analysis indicates that the term "Todo" in the list entries (especially on lines 54–55) might be more clear as "To-do" (with a hyphen) if referring to tasks. Please verify that the current usage aligns with the project's branding guidelines and adjust if needed.
🧰 Tools
🪛 LanguageTool
[grammar] ~54-~54: It appears that a hyphen is missing in the noun “To-do” (= task) or did you mean the verb “to do”?
Context: ...ps://todo-small.bitplatform.cc): .NET 9 Todo demo app with smaller download footprin...(TO_DO_HYPHEN)
[grammar] ~55-~55: It appears that a hyphen is missing in the noun “To-do” (= task) or did you mean the verb “to do”?
Context: ...://todo-offline.bitplatform.cc): .NET 9 Todo demo app with ef-core & sqlite (Azure s...(TO_DO_HYPHEN)
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
📒 Files selected for processing (1)
README.md(1 hunks)
🧰 Additional context used
🪛 LanguageTool
README.md
[grammar] ~54-~54: It appears that a hyphen is missing in the noun “To-do” (= task) or did you mean the verb “to do”?
Context: ...ps://todo-small.bitplatform.cc): .NET 9 Todo demo app with smaller download footprin...
(TO_DO_HYPHEN)
[grammar] ~55-~55: It appears that a hyphen is missing in the noun “To-do” (= task) or did you mean the verb “to do”?
Context: ...://todo-offline.bitplatform.cc): .NET 9 Todo demo app with ef-core & sqlite (Azure s...
(TO_DO_HYPHEN)
🪛 markdownlint-cli2 (0.17.2)
README.md
45-45: Table column count
Expected: 6; Actual: 2; Too few cells, row will be missing data
(MD056, table-column-count)
46-46: Table column count
Expected: 6; Actual: 2; Too few cells, row will be missing data
(MD056, table-column-count)
🔇 Additional comments (1)
README.md (1)
48-57: Consistent Renumbering of Showcase LinksThe numbered list entries (lines 48–57) have been updated to reflect the new ordering as described in the PR objectives, with the Sales entry now at number 9. Please double-check that all URLs and descriptions are correct.
🧰 Tools
🪛 LanguageTool
[grammar] ~54-~54: It appears that a hyphen is missing in the noun “To-do” (= task) or did you mean the verb “to do”?
Context: ...ps://todo-small.bitplatform.cc): .NET 9 Todo demo app with smaller download footprin...(TO_DO_HYPHEN)
[grammar] ~55-~55: It appears that a hyphen is missing in the noun “To-do” (= task) or did you mean the verb “to do”?
Context: ...://todo-offline.bitplatform.cc): .NET 9 Todo demo app with ef-core & sqlite (Azure s...(TO_DO_HYPHEN)
2 participants
closes #9837
Summary by CodeRabbit