feat: allow adding/removing usages by path#154
feat: allow adding/removing usages by path#154andrius-puksta-sensmetry wants to merge 1 commit intomainfrom
Conversation
|
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. |
But these accomplish different things: this PR simplifies creating a |
2aaf937 to
8cf5e0f
Compare
|
Could we add some integration level (i.e. |
Then call it |
Agreed that it's not obvious what "map" means. Here it refers to "mapping" (in functional programming sense) "actual" IRIs to "pseudo" IRIs. |
8cf5e0f to
76bec9a
Compare
Done |
f37f90c to
81f0ac9
Compare
…are present Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com> feat: disallow building if path dependency is detected, but allow to override this Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com> chore: update docs Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com> chore: update docs Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com> docs: add workspace docs to book Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com> docs: link to Sysml2 Pilot implementation Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com> test: add tests for adding/removing path usages and building a project with them Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com> build: bump MSRV and rust-toolchain to 1.93 Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com> fix: satisfy clippy by using if-let chains Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com> docs: don't claim all crashes are bugs Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com> docs(book): use standard warning style Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com> fix typo Signed-off-by: Andrius Pukšta <andrius.puksta@sensmetry.com>
81f0ac9 to
e2b91ce
Compare
Users want to do this anyway (at least for experimenting), so might as well accomodate them.
Changes
--pathargument toaddandremovethat allows adding/removing path usages by converting them tofile://URL. Recommend using workspaces instead of paths.--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)