Skip to content

Conversation

@brunovdias
Copy link

No description provided.

@brunovdias brunovdias requested review from marhcouto June 3, 2024 18:39
@brunovdias brunovdias changed the title (refactor): removed ports mapping and simplified overall (refactor): removed ports mapping and simplified overall FSF-3930 Jun 3, 2024
@notion-workspace
Copy link

Copy link
Collaborator

@marhcouto marhcouto left a comment

Choose a reason for hiding this comment

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

Aquilo que está nos comentarios

docker compose up
```

In case you want to do some work inside the container, by using the Dev Containers extension in VSCode the process becomes very simple, since the system is already setup for that.
Copy link
Collaborator

Choose a reason for hiding this comment

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

In case you want to do some work inside the container, by using the Dev Containers extension in VSCode the process becomes very simple, since there is a .devcontainer which contains the configuration for a development container for that purpose.


In case you want to do some work inside the container, by using the Dev Containers extension in VSCode the process becomes very simple, since the system is already setup for that.

Inside the devcontainer, in order to run the simulator, simply use one of the launch files present in the [launch folder](../launch/):
Copy link
Collaborator

Choose a reason for hiding this comment

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

Isto é bastante óbvio

Copy link
Collaborator

Choose a reason for hiding this comment

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

Acho que podemos tirar tudo para baixo da linha 20 e mudar o nome do tutorial para docker usage

@sonarqubecloud
Copy link

sonarqubecloud bot commented Jun 4, 2024

Quality Gate Passed Quality Gate passed

Issues
0 New issues
0 Accepted issues

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@marhcouto marhcouto merged commit 7dae615 into master Jun 7, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants