-
-
Notifications
You must be signed in to change notification settings - Fork 28
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
collect supplements in issue contract #231
collect supplements in issue contract #231
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #231 +/- ##
========================================
+ Coverage 17.6% 18.1% +0.6%
========================================
Files 37 37
Lines 7535 7548 +13
========================================
+ Hits 1323 1369 +46
+ Misses 6212 6179 -33
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. |
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.
You can just do
supplements: confined_bset![
Supplement::new(ContentRef::Schema(schema.schema_id()), schema.developer.clone()),
...
]
Why are you merging master? Just do rebases |
Sorry....I always use the |
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.
Sorry, just understood that in this PR instead of gathering supplements from the stash you create some pseudo-supplements out of arbitrary cosnensus data.
Approach NACK. Please either collect supplements from the stash - or close the PR
I agree with you, the first idea is to collect supplements from the stash, but the question is: do we have a stash when we issue a contract? (In the |
When you issue contract you need to collect just supplements for interfaces, schema and implementation. There is no "genesis supplement" which can be found in the stash. You also need to keep in mind that right now there is no supplements which exist - this functionality is there only as a way to add new things in the future. |
Description:
add collect supplements feature for the
issue_contract
, includes the following Supplement instance:genesis supplement
iface supplement
iimpl supplement
schema supplement