Skip to content

Added ROS Noetic Docker File#4

Closed
Srindot wants to merge 1 commit intosoham2560:mainfrom
Srindot:main
Closed

Added ROS Noetic Docker File#4
Srindot wants to merge 1 commit intosoham2560:mainfrom
Srindot:main

Conversation

@Srindot
Copy link
Copy Markdown

@Srindot Srindot commented Apr 3, 2025

No description provided.

@soham2560 soham2560 self-requested a review April 3, 2025 17:19
@soham2560 soham2560 added the enhancement New feature or request label Apr 3, 2025
@soham2560
Copy link
Copy Markdown
Owner

This seems to be tackling the same feature as #3 , maybe you guys can coordinate so that less effort is wasted, or you can put a PR targetting his branch or other way around, you decide

@soham2560
Copy link
Copy Markdown
Owner

can you also add a test command in the PR description so reviewers can test this easily? or provide a sample .devcontainer conifg

@rtarun1
Copy link
Copy Markdown
Collaborator

rtarun1 commented Apr 4, 2025

@soham2560 , the issue is fixed! we can close this PR

@soham2560
Copy link
Copy Markdown
Owner

okay sounds good, @Srindot if you want to add any feature to the other PR (since I was able to build this branch my own PC), you can open a PR targetting the other one, or if its already merged, directly to main

closing for now, thanks for the attempt though!

@soham2560 soham2560 closed this Apr 5, 2025
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