Untrack agent instructions, clean README status wording - #1
Merged
Conversation
CLAUDE.md is local-only by convention and its content is fully covered by hardware/DESIGN.md, so it is removed without relocation. README: drop (WIP) from the title, add an accurate status line, remove the CLAUDE.md advertisement, fix the structure listing to match the repo (no firmware/ or images/ dirs), point the OpenDrone link at the incutec-hw org. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
🟢 Ready to approve
Changes are low-risk (docs + ignore rules) with only minor wording/pattern anchoring suggestions noted.
This review doesn't count toward merge requirements. Sign up for the private preview to control whether Copilot approvals count.
Pull request overview
This PR removes local-only agent instruction artifacts from version control and updates the repository’s README to reflect the current hardware-only state and correct project links/structure.
Changes:
- Removed
CLAUDE.mdfrom the repo and added agent-instruction paths to.gitignore. - Updated
README.mdtitle/status wording, removedCLAUDE.mdreferences, and corrected the repository tree + OpenDrone link.
File summaries
| File | Description |
|---|---|
| README.md | Refreshes project status text, wording, repo tree, and OpenDrone link to match current repo contents. |
| CLAUDE.md | Deletes the committed agent-instructions file (intended to be local-only). |
| .gitignore | Adds ignore rules for local agent instruction files/directories. |
Review details
- Files reviewed: 2/3 changed files
- Comments generated: 1
- Review effort level: Lite
We're testing this review assessment. Please use 👍 or 👎 to tell us if it's correct.
| The module is **fully standalone**: it has its own GPS, its own BLE/WiFi radio, and its own processor. Just wire 5V + GND from the drone. No flight controller integration needed for Remote ID compliance. | ||
|
|
||
| **Optional GPS passthrough:** Wire the module's TX pad to an FC UART RX — the module relays NMEA data from its GPS to Betaflight/iNAV for GPS Rescue and OSD coordinates. The FC gets a GPS source for free. | ||
| **Optional GPS passthrough:** Wire the module's TX pad to an FC UART RX and the module relays NMEA data from its GPS to Betaflight/iNAV for GPS Rescue and OSD coordinates. The FC gets a GPS source for free. |
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.
CLAUDE.md is local-only by convention but was committed here. Its technical content is fully covered by the existing hardware/DESIGN.md, so it is removed without relocation.
🤖 Generated with Claude Code