Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add Docker configuration for turtlebot3 environment #1050

Merged
merged 3 commits into from
Feb 21, 2025
Merged

Conversation

Woojin-Crive
Copy link
Contributor

Description

This pull request merges the support-docker branch into the main branch, introducing Docker support to the turtlebot3 project. The changes in this branch aim to simplify the setup process and streamline the development environment through containerization.

Summary of Changes

  • Dockerfile and docker-compose configuration file Added:
    A new Dockerfile is provided to build a containerized environment for turtlebot3. This ensures that users can quickly and consistently set up the required build environment.

Motivation

Many users and contributors have expressed interest in a containerized setup that can simplify dependency management and environment configuration. By integrating Docker support, we reduce setup time and increase the reproducibility of builds and tests. This change helps new contributors get started quickly and makes the project more accessible.

Testing & Validation

  • Local Testing:
    The Docker container was built and run locally, and all essential functionalities were verified within the containerized environment.

Request for Review

Please review the changes and provide any feedback or suggestions. Your input is welcome to ensure that this new workflow integrates smoothly with our ongoing development process.

@yun-goon yun-goon self-requested a review February 13, 2025 09:00
@yun-goon yun-goon added the enhancement New feature or request label Feb 13, 2025
@yun-goon yun-goon self-assigned this Feb 13, 2025
Copy link
Contributor

@yun-goon yun-goon left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

image
@Woojin-Crive I confirmed that the topics are received properly even in Docker! Thank you for your contribution!

Copy link
Member

@robotpilot robotpilot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM
@Woojin-Crive FYI, We have a docker account, I'll let you know later.
https://hub.docker.com/orgs/robotis/

@robotpilot robotpilot removed the request for review from sunghowoo February 21, 2025 11:26
@robotpilot robotpilot merged commit fe99c9b into main Feb 21, 2025
9 checks passed
@robotpilot robotpilot deleted the support-docker branch February 21, 2025 11:27
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants