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

improve configuration details on readme #5

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

pxalcantara
Copy link

Improve the README file with more details about the ros2.code-workspace configuration and adding some possibility to use different workspaces.

@pxalcantara pxalcantara added the documentation Improvements or additions to documentation label Apr 22, 2021
@pxalcantara pxalcantara self-assigned this Apr 22, 2021
Copy link
Contributor

@mateus-amarante mateus-amarante left a 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
Comment on lines 33 to 41
```json
// FOLDERS
"folders": [
{
"path": "foxy_ws"
}
],
```

Copy link
Contributor

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:
Copy link
Contributor

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
Copy link
Contributor

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.

@pxalcantara
Copy link
Author

Nice addition!

Just check my comments and fix the CI errors. You can easily check this in vscode with the orise's workspace.

Done!

Copy link

@ivan1993br ivan1993br left a comment

Choose a reason for hiding this comment

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

Nice!

@caioaamaral
Copy link

Nice Work

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants