Skip to content

Revise README for clarity and updated links#3429

Merged
beastoin merged 1 commit intomainfrom
beastoin-patch-1
Nov 14, 2025
Merged

Revise README for clarity and updated links#3429
beastoin merged 1 commit intomainfrom
beastoin-patch-1

Conversation

@beastoin
Copy link
Copy Markdown
Collaborator

Removed redundant closing remarks and updated links for app development.

Removed redundant closing remarks and updated links for app development.
@beastoin beastoin merged commit 4f5e8ec into main Nov 14, 2025
1 check passed
@beastoin beastoin deleted the beastoin-patch-1 branch November 14, 2025 04:22
Copy link
Copy Markdown
Contributor

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request improves the main README by removing redundant text and updating the 'In this repo' section with more descriptive, relative links. The changes enhance clarity and maintainability. I've found one issue with a broken link that should be addressed.

Comment thread README.md
- [ai personas (web)](https://github.com/BasedHardware/omi/tree/main/personas-open-source)
- [SDKs](sdks)
- [omi device](omi) *nrf\* chips, zephyr (c/c++)*
- [omi glass](omiglass) *esp32-s3, c/c++*
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

high

The link to omi glass is broken due to incorrect casing. The directory is named omiGlass, but the link points to omiglass. This will not work on case-sensitive file systems like the one GitHub uses. Please correct the link to match the directory name.

Suggested change
- [omi glass](omiglass) *esp32-s3, c/c++*
- [omi glass](omiGlass) *esp32-s3, c/c++*

Glucksberg pushed a commit to Glucksberg/omi-local that referenced this pull request Apr 28, 2026
Removed redundant closing remarks and updated links for app development.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant