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
However, when a viash build or viash ns build is run, the resulting .config.vsh.yamlshould now contain the following info, because the resource files are now available relative to the .config.vsh.yaml.
This could be solved in a variety of ways. Either we could:
Not do any resource path resolution when running viash config view or viash ns list
Rename a resource's parent field to uri and make it store the full uri instead of the parent directory uri. That way, we can let the path field simply retain the unmodified original data.
Use different classes to denote whether the resource has not been resolved (string) or has been resolved (string and uri to parent dir / resource itself).
Confirmation
I have searched the existing issues to make sure this is not a duplicate.
I have provided clear and concise information about the bug.
Additional context
No response
The text was updated successfully, but these errors were encountered:
What happened?
Viash 0.9 no longer outputs the parent of a resource when running
viash config view
Steps to reproduce
Create a config
src/foo/config.vsh.yaml
with contents:Run
viash config view
:According to the resulting
viash config view
, it appears as if theREADME.md
is relative to the foo component, which is not the case.With Viash < 0.9, the output of
viash config view
is the following:I understand and agree that the parent attribute is not something that we want to output.
Expected behavior
Having said that, I would expect the output of
viash config view
to contain file paths that can actually be resolved.That is, the output of
viash config view
andviash ns list
show the original values stored in the viash config:However, when a
viash build
orviash ns build
is run, the resulting.config.vsh.yaml
should now contain the following info, because the resource files are now available relative to the .config.vsh.yaml.Relevant log output
No response
Version
0.9.0-RC2
Possible solution
This could be solved in a variety of ways. Either we could:
parent
field touri
and make it store the full uri instead of the parent directory uri. That way, we can let thepath
field simply retain the unmodified original data.Confirmation
Additional context
No response
The text was updated successfully, but these errors were encountered: