[Doc] Clarify README install and quick start guidance#1453
[Doc] Clarify README install and quick start guidance#1453minggangw merged 2 commits intoRobotWebTools:developfrom
Conversation
There was a problem hiding this comment.
Pull request overview
Clarifies install vs. quick-start guidance in the project README and the npmjs-specific README, aiming to reduce confusion around environment setup, install sources (npm vs GitHub), and message generation.
Changes:
- Adds explicit ROS 2 environment “source” guidance and separates installation paths (npm dependency vs repo quick start).
- Reworks GitHub install instructions into concrete
npm install RobotWebTools/rclnodejs#...commands. - Expands documentation links (adds tutorials link in the npmjs README) and refines message-generation wording.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| scripts/npmjs-readme.md | Clarifies prerequisites (sourcing ROS 2), GitHub install command, docs links, and message-generation guidance for the npm page. |
| README.md | Reorganizes installation/quick-start instructions, updates anchors/headings, and refines docs guidance. |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
scripts/npmjs-readme.md
Outdated
| rclnodejs ships with prebuilt native binaries for common Linux configurations since `v1.5.2`, eliminating the need for compilation during installation. This applies to supported Linux environments when installing from npm or GitHub. | ||
|
|
There was a problem hiding this comment.
The statement that prebuilt binaries apply when installing from GitHub appears incorrect. scripts/install.js only checks for a local prebuilds/<platform>-<arch>/... file and does not download prebuilds; since the prebuilds/ directory is not present in the repo, npm install RobotWebTools/rclnodejs#... will fall back to building from source. Consider limiting this claim to npm installs (published tarballs) or documenting that GitHub installs build from source unless the branch includes prebuilds/ artifacts.
| rclnodejs ships with prebuilt native binaries for common Linux configurations since `v1.5.2`, eliminating the need for compilation during installation. This applies to supported Linux environments when installing from npm or GitHub. | |
| rclnodejs ships with prebuilt native binaries for common Linux configurations since `v1.5.2`, eliminating the need for compilation during installation when installing from npm. These prebuilds are bundled in the published npm package tarballs. | |
| GitHub installs (for example, `npm install RobotWebTools/rclnodejs#<branch>`) build from source by default unless that branch or tag includes `prebuilds/` artifacts for your platform. |
README.md
Outdated
| ### Prebuilt Binaries | ||
|
|
||
| rclnodejs ships with prebuilt native binaries for common Linux configurations since `v1.5.2`, eliminating the need for compilation during installation. This significantly speeds up installation and reduces dependencies. | ||
| rclnodejs ships with prebuilt native binaries for common Linux configurations since `v1.5.2`, eliminating the need for compilation during installation. This applies to supported Linux environments when installing from npm or GitHub. |
There was a problem hiding this comment.
This sentence implies GitHub installs get prebuilt native binaries, but the current install flow (scripts/install.js) only uses prebuilds if a prebuilds/ directory exists locally and does not fetch them. Since prebuilds/ is not present in the repo, npm install RobotWebTools/rclnodejs#... will typically build from source. Please adjust the wording to avoid promising prebuilt binaries for GitHub installs (or explain the required conditions).
| rclnodejs ships with prebuilt native binaries for common Linux configurations since `v1.5.2`, eliminating the need for compilation during installation. This applies to supported Linux environments when installing from npm or GitHub. | |
| rclnodejs ships with prebuilt native binaries for common Linux configurations since `v1.5.2`, eliminating the need for compilation during installation when a matching prebuilt is available. In typical setups, this applies to supported Linux environments when installing from npm. When installing directly from GitHub (for example using `npm install RobotWebTools/rclnodejs#<branch>`), the package will usually be built from source unless your install process provides compatible prebuilt binaries locally. |
Fix: #1452