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

docs(readme): add --rm to Docker example #101

Merged
merged 1 commit into from
Feb 10, 2023
Merged

Conversation

glensc
Copy link
Contributor

@glensc glensc commented Feb 9, 2023

there's no need to keep the used container around.

ideally one could persist the used data paths, i.e profile data.

-v $(pwd)/state/config:/root/.config -v $(pwd)/state/cache:/root/.cache

but I haven't investigated so far what paths need to persist between runs.

there's no need to keep the used container around
@AlexanderMaxRanabel
Copy link
Contributor

good idea

@fathyb fathyb changed the title docker: remove container after use docs(readme): add --rm to Docker example Feb 10, 2023
@fathyb fathyb merged commit 191d45d into fathyb:main Feb 10, 2023
@fathyb
Copy link
Owner

fathyb commented Feb 10, 2023

Thanks!

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