Skip to content
This repository was archived by the owner on Nov 15, 2023. It is now read-only.

srml-contract: Rename Contract::create to instantiate #3645

Merged
merged 6 commits into from
Sep 19, 2019
Merged

srml-contract: Rename Contract::create to instantiate #3645

merged 6 commits into from
Sep 19, 2019

Conversation

expenses
Copy link
Contributor

@expenses expenses commented Sep 19, 2019

See #2828. I'm not sure how much of the documentation needs updating.

@parity-cla-bot
Copy link

It looks like @expenses signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

1 similar comment
@parity-cla-bot
Copy link

It looks like @expenses signed our Contributor License Agreement. 👍

Many thanks,

Parity Technologies CLA Bot

@xlc
Copy link
Contributor

xlc commented Sep 19, 2019

you have to bump spec_version (and set impl_version to the same number) as this is a spec breaking change

@expenses
Copy link
Contributor Author

@xlc Done 🙂

Verified

This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
Copy link
Contributor

@pepyakin pepyakin left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks good! Although there are still some things to do in #2828, e.g. rename ext_create -> ext_instantiate, update docs, etc

@pepyakin pepyakin merged commit 37bc8c5 into paritytech:master Sep 19, 2019
en pushed a commit to en/substrate that referenced this pull request Sep 24, 2019
* Rename Contract::create to Contract::instantiate

* Rename some documentation items

* fixed node executor test

* bumped impl_version

* increased spec_version
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

5 participants