-
Notifications
You must be signed in to change notification settings - Fork 0
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
improve configuration details on readme #5
base: main
Are you sure you want to change the base?
Conversation
cbc51cd
to
e1199bc
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice addition!
Just check my comments and fix the CI errors. You can easily check this in vscode with the orise's workspace.
README.md
Outdated
```json | ||
// FOLDERS | ||
"folders": [ | ||
{ | ||
"path": "foxy_ws" | ||
} | ||
], | ||
``` | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actually, you can do it more easily through the user interface. You could simply reference this tutorial instead.
README.md
Outdated
@@ -10,6 +10,35 @@ To work with containers, this package provides the helper script `setup_workspac | |||
|
|||
Run `setup_workspace.sh --help` for more details about the script usage. | |||
|
|||
If you're using [ORise container](https://github.com/orise-robotics/ros_ws), it assumes that the folder `/home/orise` will be used as your woskspace to develop using ROS 2 and also, your VSCode workspace. If you want to use different workspaces inside the container, just copy the files `.vscode-format` and `ros2.code-workspace` to the root of your ROS workspace and redefine the `COLCON_WORKSPACE_FOLDER` variable. For example, let's assume that you want to use a ROS 2 woskpace called *foxy_ws*, this folder should have: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Markdownlint complains about this line size. Consider splitting them into two lines (a single enter do not jump lines in the rendered output)
README.md
Outdated
And you must redefine the `COLCON_WORKSPACE_FOLDER` variable: | ||
|
||
```sh | ||
$ export COLCON_WORKSPACE_FOLDER=/home/orise/foxy_ws |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Markdownlint is complaining about this line. Check why.
cec69f9
to
7a02ea7
Compare
Done! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
Nice Work |
Improve the README file with more details about the
ros2.code-workspace
configuration and adding some possibility to use different workspaces.