Skip to content

Fix docker invocation in README#54

Open
mtausig wants to merge 1 commit intoinsidersec:masterfrom
mtausig:patch-1
Open

Fix docker invocation in README#54
mtausig wants to merge 1 commit intoinsidersec:masterfrom
mtausig:patch-1

Conversation

@mtausig
Copy link

@mtausig mtausig commented Jun 28, 2021

What was a problem?

The docker version should not be run with the --rm, as this would delete the report right away.

How this PR fixes the problem?

Remove the option from the README

Check lists (check x in [ ] of list items)

  • Test passed (soon)
  • Coding style (indentation, etc)

Additional Comments (if any)

Optimally, there should be solution to store the report in a different folder.

The docker version should not be run with the `--rm`, as this would delete the report right away.
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.

1 participant