Skip to content

Commit 0aeae72

Browse files
authored
Remove outdated Dockerfile (#1262)
This PR removes an outdated Dockerfile and replaces it with documentation pointing users to official ROS Docker images. The change simplifies the project by eliminating maintenance of a custom Docker setup while providing clear guidance for containerized builds. Key changes: - Complete removal of the custom Dockerfile - Addition of Docker guidance in README and npmjs documentation pointing to official ROS images Fix: #1261
1 parent 19c9c31 commit 0aeae72

File tree

3 files changed

+4
-51
lines changed

3 files changed

+4
-51
lines changed

Dockerfile

Lines changed: 0 additions & 51 deletions
This file was deleted.

README.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -42,6 +42,8 @@ rclnodejs.init().then(() => {
4242
- [Node.js](https://nodejs.org/en/) version >= 16.13.0
4343
- [ROS 2 SDK](https://docs.ros.org/en/jazzy/Installation.html) - **Don't forget to [source the setup file](https://docs.ros.org/en/jazzy/Tutorials/Beginner-CLI-Tools/Configuring-ROS2-Environment.html#source-the-setup-files)**
4444

45+
> **Docker:** Use official ROS images from [hub.docker.com/\_/ros](https://hub.docker.com/_/ros) for containerized builds.
46+
4547
### Install rclnodejs
4648

4749
```bash

scripts/npmjs-readme.md

Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -23,6 +23,8 @@ rclnodejs.init().then(() => {
2323
- [Node.js](https://nodejs.org/en/) version >= 16.13.0
2424
- [ROS 2 SDK](https://docs.ros.org/en/jazzy/Installation.html) - **Don't forget to [source the setup file](https://docs.ros.org/en/jazzy/Tutorials/Beginner-CLI-Tools/Configuring-ROS2-Environment.html#source-the-setup-files)**
2525

26+
> **Docker:** Use official ROS images from [hub.docker.com/\_/ros](https://hub.docker.com/_/ros) for containerized builds.
27+
2628
### Install rclnodejs
2729

2830
```bash

0 commit comments

Comments
 (0)