canonical / canonical/data-science-stack

Define the manifests path as a variable in `config.py`

Open
#61 1 comment 0 reactions 0 assignees View on GitHub
enhancement
Dominant language
Python
Stars
37
Forks
10
PR merge metrics
No merged PRs in 30d

Description

### Why it needs to get done

to define the manifests path in one place and reuse it for all commands
it was first discussed in [this comment](https://github.com/canonical/data-science-stack/pull/43#discussion_r1517241851)

### What needs to get done

Move the definition of the manifests path to a variable in `config.py` and reuse this variable in all the commands `.py` files that interact with the manifests path.

in `config.py` do something like:
```
MANIFEST_TEMPLATES_LOCATION = Path("./manifest_templates")
ABSOLUTE_MANIFEST_TEMPLATES_LOCATION = Path(__file__).parent / MANIFEST_TEMPLATES_LOCATION
```

### When is the task considered done

manifests path definition is refactored as described and PR is merged

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.