-
Notifications
You must be signed in to change notification settings - Fork 41
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
Punch List: Package #13
Comments
@kestewart to think about package-file-name as relationship. |
A package doesn't have a file name, because what Once what a For example, a package In conclusion, my preference would be to omit a file name property on |
@seabass-labrax that's my preference as well for the same reasoning, @kestewart raised the concern so let's wait for her feedback before closing this item. |
From the discussion on the 15 April extended tech call, there was general consensus that this should be a relationship. This provides additional information and is a more accurate representation of how the package is used. There does, however, create some additional complexity by requiring a relationship and isn't compatible with 2.X where it is just a string property + a hash property. On the call we discussed a need to provide good documentation on how to use relationships for the package file to at least partially address the complexity concern. |
On the 15 April extended tech call we discussed a couple of different use cases for the package file where the file may be located in multiple location or the actual destination of the package file isn't know. I would propose adding a couple of new relationships to help cover these use cases:
|
@iamwillbar - am closing this issue as all the topics in the punch list have been discussed. Please reopen if you disagree. |
This is a punch list of open questions from the 2021-12-07 Tech Team meeting. Please comment on this issue with any discussion, proposed answers, or additional questions you have:
The text was updated successfully, but these errors were encountered: