Skip to content
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

Fix clippy error: Manual implementation of Option::map #717

Merged
merged 1 commit into from
Mar 5, 2021

Conversation

cmichi
Copy link
Collaborator

@cmichi cmichi commented Mar 5, 2021

This the latest merge from https://github.com/paritytech/scripts/ fails building a new container due to our master being broken with this clippy error.

@cmichi cmichi force-pushed the cmichi-make-master-green-again branch from a02a277 to f14d2da Compare March 5, 2021 07:31
@cmichi cmichi requested a review from Robbepop March 5, 2021 07:45
@cmichi cmichi merged commit 05e569b into master Mar 5, 2021
@cmichi cmichi deleted the cmichi-make-master-green-again branch March 5, 2021 07:59
cmichi pushed a commit that referenced this pull request May 10, 2021
* Add `engine` crate

* Add `env_types` crate

* Adapt `env`, `lang` and `storage`

* Adapt examples

* Adapt CI

* Symlink license and readme

* Throw `TypedEncoded` out of `engine`

* Improve Erc20

* Bump versions to rc3

* Fix clippy error: Manual implementation of `Option::map` (#717)

* Implement comments

* Fix yml

* Improve structure

* Add tests

* Fix function signature

* Get rid of `engine`s singleton

* Revert instantiate stuff

* Implement review comments

* Make `Storage` non-generic

* Improve API for emmitted events

* Migrate to `panic_any`

* Clean up import

* Import `panic_any`

* Implement comments

* Fix param

* Use type

* Store balances in chain storage

* Fix tests

* Use individual storage per contract

* Implement comments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants