docs: remove dependency notes section from README#172
Conversation
Co-Authored-By: Rahul Chalamala <22563365+rchalamala@users.noreply.github.com>
🤖 Devin AI EngineerI'll be helping with this pull request! Here's what you should know: ✅ I will automatically:
Note: I can only respond to comments from users who have write access to this repository. ⚙️ Control Options:
|
| @@ -5,7 +5,3 @@ Made with ❤️ by [Rahul](https://github.com/rchalamala/), [Eric](https://gith | |||
| In addition, thanks to [Armeet](https://github.com/armeetjatyani/) and others for suggestions/contributions! | |||
|
|
|||
| Favicon art by Audrey Wong. | |||
There was a problem hiding this comment.
🚩 Removing documentation guard for non-obvious peer dependencies
The deleted section explicitly warned maintainers that preact and @preact/signals are peer dependencies of @schedule-x/* and must not be removed, despite never appearing in src/ imports. These dependencies are still present in package.json:12,21 and are required at runtime by @schedule-x/calendar (confirmed in package-lock.json:2931-2932). Without this note, a future contributor running a dependency audit or cleanup could reasonably conclude these are unused and remove them, breaking the Schedule-X calendar in src/Planner.tsx. Consider keeping this note somewhere (e.g., as a comment in package.json or in a contributing guide) if the README isn't the right place.
Was this helpful? React with 👍 or 👎 to provide feedback.
There was a problem hiding this comment.
This removal was explicitly requested by the repo owner.
Deploying with
|
| Status | Name | Latest Commit | Preview URL | Updated (UTC) |
|---|---|---|---|---|
| ✅ Deployment successful! View logs |
caltech-dev | 7c900f8 | Commit Preview URL Branch Preview URL |
Jun 22 2026, 06:37 AM |
Summary
Remove the "Dependency notes" section from
README.md(thepreact/@preact/signalspeer-dependency note for Schedule-X).Link to Devin session: https://app.devin.ai/sessions/c64519508c034ff78cfbdd2b6c217bbd
Requested by: @rchalamala