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

Add question and selection of optional container/conda cache dirs #46

Draft
wants to merge 13 commits into
base: main
Choose a base branch
from

Conversation

jfy133
Copy link
Collaborator

@jfy133 jfy133 commented Jan 24, 2024

This is to close #20

@jfy133 jfy133 marked this pull request as draft January 24, 2024 20:56
@jfy133
Copy link
Collaborator Author

jfy133 commented Jan 24, 2024

Current issues:

  • How to get both the variable AND the path to print as options
  • create_cachedir_options() just adds the var name to a list if it's true, so need to maybe replace check_var_in_env to export a dict with var, path pair
  • How to return output of create_container_scope to pass to writing, as now exporting two sets of options (container engine itself + options; and now cache dir)

@jfy133
Copy link
Collaborator Author

jfy133 commented Jan 30, 2024

Also to add: if manual selection made, check directory exists

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.

Infrastructure config - autodetect: is a cachedir variable already set, for resuse
1 participant