You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
The way we're currently using the sqlite db requires us to package it up with the python package. This is generally not a good practice .
It might be cleaner to put the db in a location such as next to our other com.pieces.os contents, but afaik poetry (that is our current build manager) does not currently support setuptools' data_files.
Alternatively we might want to allow cli's user to specify it themselves wherever they see fit, e.g. with some config file, or by supplying the path as an option in the cli.
Not sure what the optimal course of action for this is here, but we should be aware of this, hence I'm raising this issue.
The text was updated successfully, but these errors were encountered:
The way we're currently using the sqlite db requires us to package it up with the python package. This is generally not a good practice .
It might be cleaner to put the db in a location such as next to our other com.pieces.os contents, but afaik poetry (that is our current build manager) does not currently support setuptools' data_files.
Alternatively we might want to allow cli's user to specify it themselves wherever they see fit, e.g. with some config file, or by supplying the path as an option in the cli.
Not sure what the optimal course of action for this is here, but we should be aware of this, hence I'm raising this issue.
The text was updated successfully, but these errors were encountered: