-
Notifications
You must be signed in to change notification settings - Fork 15
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
List/Load Projects #4
Comments
Would prefer
The only problem with that is, what do you do about the current working directory? E.g. you are in |
Throw an error message about loading a project in the directory that already contains Or is |
Yeah, |
Could we throw an error when loading a project that is not named the same as the current |
|
If you are in If |
Happy to try that. |
Just looked into the toml files, and there is no record of the project name in the toml files. This means we cant compare the desired project name to load, to what exists on disk. How should we proceed with this? |
Remind me again what the problem was loading the project based on the current working dir, and having a |
If you want a folder named different from current working dir. For example "my_project" folder, but load "My Project". |
I see what you mean, and it should be possible, but perhaps the CLI isn't the best tool for it. I'm a little weary of letting the CLI become too feature rich, and would rather it remain a fallback for when a more feature complete GUI or web interface fails. Basically, I think the effort is just but that it may yield a greater return of investment elsewhere. I'm not entirely confident about that and could see it both ways. We could also look at making the CLI more central as a CLI would be quicker to develop, as GUIs invite a lot of complexity. What do you think about that? |
Yeah, I did tend to find that I need to change both avalon-setup and avalon-core, when doing these changes. The CLI in avalon-setup seems to be a mirror of the inventory in avalon-core. I personally dont have enough of an overview of Avalon to know where and how this should proceed, but I'm up for working on it. Currently though I have everything for this issue ready, except that you can overwrite data on disk if you load a different project. This was doable before as well, but required you to change the name of the current working directory instead of just a flag to |
Yes, let's do that.
Yes, not a mirror per-se, but rather forwarding requests to it. What I've tried to do is keep functionality as local where possible; the inventory in core is is used internally by the core library, and the CLI is like a GUI for it. Calling upon it where necessary. I would consider making it
|
For completeness, to modify the argument for |
Just getting my bearings, it's not the most obvious system, but it's all driven by environment variables up till the current project. From there, any command is relative this project. For example, I think that might be what you're looking for. Basically, if no path is passed via the CLI, look at the current working directory. |
Goal
To make it easier to list and load projects.
Motivation
avalon --load
, the name is taken from the current working directory, so there is no way of loading a differently named project.avalon --load
it would be nice to display available projects to load.Implementation
avalon --projects
. Or some other flag?avalon --load
accept a project name input, so loading a project will beavalon --load myProject
. This input could be optional to allow existing behaviour of loading the name of the current working directory.Lastly could also have the command be something like
avalon --load --project_name myProject
. Although personally I feel its unnecessarily long.Loading .inventory.toml and .config.toml.. 'mySecondProject' not found, try --init to start a new project.
Suggestion:
Loading .inventory.toml and .config.toml.. 'mySecondProject' not found, try --init to start a new project, or load a project from the database. Projects: - myProject - mySecondProject
The text was updated successfully, but these errors were encountered: