-
Notifications
You must be signed in to change notification settings - Fork 2
feat: allow adding usages by path #154
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
base: main
Are you sure you want to change the base?
Conversation
…are present Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
|
So while using file urls for quick experimentation is fine, it's non-portable so generally bad practise and should not be encouraged. As such I would prefer not to have a nice and simple flag like If we are to use a |
I do think we need something that is (equivalently) easy to use for learning purposes, and A complex alternative option would be to have the Sysand client maintain some sort of "local index" that the user can either install, or register projects to. This is sort of doable already (using |
I strongly believe that if we allow adding usages with path argument, this should be a hard error because URIs are global, so such kpar is (almost surely) unusable outside of the computer on which it was created. Also, in the documentation we probably should recommend using workspaces instead. |
…override this Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
But these accomplish different things: this PR simplifies creating a |
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
Users want to do this anyway (at least for experimenting), so might as well accomodate them.
Changes
--pathargument toaddthat allows adding path usages by converting them tofile://URL. Recommend using workspaces.--allow-path-usageis passed (in that case a warning is still produced)initinstead ofnew(sysand newwas mentioned in AoSysML, so this at least gives users something actionable)