Conversation
BundleMonNo change in files bundle size Unchanged groups (1)
Final result: ✅ View report in BundleMon website ➡️ |
There was a problem hiding this comment.
Pull request overview
Updates the project README to provide clearer setup instructions and a more complete description of the app’s capabilities.
Changes:
- Rewrites the quick start/setup sections into clearer Installation/Development/Build instructions.
- Adds a detailed Features section describing the two main tabs and graph visualization modes.
- Documents high-level architecture (Clean Architecture + CompositionRoot) and i18n workflows.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 1 comment.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
|
|
||
| - You can use `.env` variables within the React app: `const value = import.meta.env.NAME;` | ||
| ``` | ||
| $ yarn extract-pot |
There was a problem hiding this comment.
The i18n instructions run yarn extract-pot and then yarn localize, but localize already runs update-po, which itself calls extract-pot (see package.json scripts). This makes the documented steps redundant and can slow down workflows; consider documenting just yarn localize, or adjusting the scripts so each step is necessary.
| $ yarn extract-pot |
Improve readme