* Check all the code with clippy

* Fix clippy warnings

* Add `--keep-symbols` flag

* Replace pwasm_utils::optimize by a simple export stripper

* Satisfy clippy

* Fix typos

Co-authored-by: Michael Müller <[email protected]>

* Fix test build errors

* Fix tests

* Rename to `--keep-debug-symbols`

* Add test for `--keep-debug-symbols`

* Fix typos

Co-authored-by: Andrew Jones <[email protected]>
Co-authored-by: Michael Müller <[email protected]>

* Restore when/then

Co-authored-by: Michael Müller <[email protected]>
Co-authored-by: Andrew Jones <[email protected]>

* Always use  for Wasm output file name

* Use `contract_artifact_name` instead of `package_name`

* Apply `cargo fmt`

* Satisfy clippy

* Suggest `binaryen` installation from GitHub on outdated version

* Fix tests

* Add CI for building contract template under Windows

* Disable `unix` specific tests

* Remove build warnings due to `dead_code` with default features

* Remove unused `binaryen` dependency

* Ensure path is always canonicalized for comparison

* Apply auto-formatting to yml

* CI: optimize caching

Co-authored-by: Denis P <[email protected]>

* Remove deprecated `check` command

* Update CHANGELOG.md

Co-authored-by: Andrew Jones <[email protected]>

Co-authored-by: Andrew Jones <[email protected]>

* Add fix

* Revert "Add fix"

This reverts commit 5c81fee5.

* Generate unique contract names for tests

* Fix test

* Implement comments

* Implement comments

error: methods with the following characteristics: (`to_*` and `self` type is `Copy`) usually take `self` by value
  --> src/workspace/profile.rs:75:22
   |
75 |     fn to_toml_value(&self) -> value::Value {
   |                      ^^^^^
   |
   = note: `-D clippy::wrong-self-convention` implied by `-D warnings`
   = help: consider choosing a less ambiguous name
   = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#wrong_self_convention

error: methods with the following characteristics: (`to_*` and `self` type is `Copy`) usually take `self` by value
   --> src/workspace/profile.rs:102:22
    |
102 |     fn to_toml_value(&self) -> value::Value {
    |                      ^^^^^
    |
    = help: consider choosing a less ambiguous name
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#wrong_self_convention

error: methods with the following characteristics: (`to_*` and `self` type is `Copy`) usually take `self` by value
   --> src/workspace/profile.rs:121:22
    |
121 |     fn to_toml_value(&self) -> value::Value {
    |                      ^^^^^
    |
    = help: consider choosing a less ambiguous name
    = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#wrong_self_convention

* Remove support for `binaryen-as-dependency`

* Add to changelog

* Fix `wasm-opt --version` parsing

* Mark stdout output and error clearer

* Implement comments

* Update readme

* Update readme

* add git commit hash in cargo-contract version

* remove todo and add version for `cargo contract` command

* It seems that I misunderstand the requirements. recover the old modify

* remove `pub`

* Improve error output

* Add installation commands

* Fix casing

* Use display instead of debug

* Improve `wasm-opt` not found error message

* Improve error message + check `wasm-opt` compatibility

* Use `regex` for parsing `wasm-opt --version`

* Apply suggestions from code review

Co-authored-by: Andrew Jones <[email protected]>

* Implement comments

* Apply cargo fmt

Co-authored-by: Andrew Jones <[email protected]>

* Check wasm optimized output file exists

* Fmt

* Add expected optimized file to error

* Use display for path

* Detect version mismatches of `parity-scale-codec`

* Detect `scale-info` mismatch as well

* Improve error message

* Clarify when a mismatch happens

* Add missing derive

* Write value actually to manifest

* Add another test

* Reduce code duplication

* Implement comments

* Derive `PartialEq` always

* Reduce code for `OptimizationPasses`

* Fix typo: zeros ➜ zeroes

* Reduce code for `OptimizationPasses`

* Add log output for optimization flags

* Support `optimization-passes` in the release profile

* Add link to Windows binary releases to Readme

* Improve failed assert message

* Account for `binaryen-rs` behavior

* Link GitHub issue in comment

* Implement comments

* Update `--help`

* Refactoring build, phase 1

* Fmt

* Extract local method for building, always optimize on build

* Add MetadataResult type

* Fmt

* Create dest wasm dir

* Execute build directly from metadata tests

* Modify existing build test to code only, since metadata test now builds all

* Fix errors after merge

* ink::test attributes in new template

* add use ink

* adding optimization-passes option

* undoing unnecessary variable name change

* remove fork changes

* adding optimization_passes to tests

* cargo fmting

* switching to enum type

* Update src/main.rs

Co-authored-by: Michael Müller <[email protected]>

* Update src/main.rs

Co-authored-by: Michael Müller <[email protected]>

* Update src/main.rs

Co-authored-by: Michael Müller <[email protected]>

* Update src/main.rs

Co-authored-by: Michael Müller <[email protected]>

* Update src/main.rs

Co-authored-by: Michael Müller <[email protected]>

* adding cli help comment

* add line so default option is more visible

* Update src/cmd/build.rs

Co-authored-by: Michael Müller <[email protected]>

* Revert "Revert me: Remove `--all-features` (#114)"

This reverts commit 4a0d352b.

* Pass verbosity flag

* Format use

* Run tests without `binaryen-as-dependency`

* Update to `scale-info` 0.6

* Fix `value used after move`

* Do not continue processing wasm on `check`

Results in `No such file` otherwise, since
no *.wasm is generated on `check`.

* Make casing consistent for `log` messages

* Clarify behavior of `check`

* Revert me: add debug output

* Upgrade `cargo_metadata` to 0.13.1

* Fix assert for target path `target/ink`

* Revert "Revert me: add debug output"

This reverts commit d2bf1dd9.

* Revert "Upgrade `cargo_metadata` to 0.13.1"

This reverts commit 723c9b29.

* Run tests only with `binaryen-as-dependency`

* Upgrade cargo-metadata and fix usages (#210)

* Only allow new contract names beginning with an alphabetic character (#219)

* Only allow contract names beginning with an alphabetic character

* Add test for contract name beginning with a number

* Add test for contract name beginning with a number

* Run tests only with `binaryen-as-dependency`

* Refactor

* Revert "Refactor"

This reverts commit defe20db

.

Co-authored-by: Andrew Jones <[email protected]>

* Only allow contract names beginning with an alphabetic character

* Add test for contract name beginning with a number

* Add test for contract name beginning with a number

* no periods in new contract names

* test for valid characters

* more readable iterator

* Build sub-contracts into sub-folders of `target/ink/`

* Reduce code duplication with util function

* Make `absolute_directory` method on `ManifestPath`

* Revert unnecessary changes

* Ensure workspace root is also the canonical path

* Use `--zero-filled-memory` for `wasm-opt`

* Assert that compiled contract template is below 3k

* Apply cargo fmt

* Remove superfluous comment

* Improve error message on `wasm-opt` error

* Specify minimum binaryen version

* Generate metadata explicitly for the contract which is build

* Improve naming and comments

* Revert me: Hotfix for funty issue

* Move path replacement logic

* Revert new line change

* Simplify for which package metadata is generated

* Change order back

* Make code clearer

* Revert me: Hotfix for funty issue

* Assert that size of resulting wasm is > 0

* Ensure optimized file is not overwritten with empty file on `not(feature = binaryen-as-dependency)`

* Fix `warning: panic message is not a string literal`

* Do not record debug info

* Update comments

* Implement Wasm validation for known issues/markers

* Add test for invalid panic import

* Add prefix to error markers

* Add test for ink! codegen error marker

* Implement improved linker error protocol

* Move validation into its own module

* Migrate tests to new linker error protocol

* Fix merge

* Improve code structure

* Fix formatting

* Improve code structure

* Remove mod indirection

* Replace `contains` with `starts_with`

* Add note regarding equivalent ink! type necessity

* Throw an error on invalid import function

* Add sunny day test

* Do not panic if no import section is found

* Update error message on decoding error

* Fix `#[warn(clippy::ptr_arg)]`

* Use either `binaryen-rs` dep or `wasm-opt` binary

* Implement reviewers suggestions

* Rename `optimization_level` to `optimization_passes`

* Revert "Rename `optimization_level` to `optimization_passes`"

This reverts commit 1fd35bcd.

* Update installation instructions

* Update readme

* Add package manager links

* Fix `#[warn(clippy::ptr_arg)]`

* Fix `#[warn(clippy::ptr_arg)]`

* Fix `#[warn(clippy::match_like_matches_macro)]`

* Fix `#[warn(clippy::type_complexity)]`

* Print warning to stderr

* Suppress output on `--quiet`

* Refactor conditional printing

* Do not generate artifacts under `target` for `contract check`

* Add test

* Use `cargo check` for `contract check`

* Replace `if` with `assert`

* Shorten code

* Minor code improvs