-
Notifications
You must be signed in to change notification settings - Fork 39
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
feat(claim.go): add bundleReference property #189
feat(claim.go): add bundleReference property #189
Conversation
Ref cnabio/cnab-spec#299 Signed-off-by: Vaughn Dice <vadice@microsoft.com>
Created time.Time `json:"created"` | ||
Modified time.Time `json:"modified"` | ||
Bundle *bundle.Bundle `json:"bundle"` | ||
BundleReference string `json:"bundleReference,omitempty"` |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Could we update the test for claims?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good call; updated.
Signed-off-by: Vaughn Dice <vadice@microsoft.com>
4ce7458
to
6a4841f
Compare
It looks like the claim schema is not up-to-date 😅 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thank you @vdice 🎉
Ok my bad the PR is here cnabio/cnab-spec#341 thank you @vdice :) |
bundleReference
property to a ClaimRef cnabio/cnab-spec#299