Skip to content
Unverified Commit 836acbff authored by Michael Müller's avatar Michael Müller Committed by GitHub
Browse files

Add ink! linting MVP stage (#431)



* Add ink! linting MVP stage

* CI: cache cargo installations; update versions

* Apply suggestions from code review

Co-authored-by: default avatarSergejs Kostjucenko <[email protected]>

* Update changelog

* Check if `dylint` requirements are installed

* Make `clippy` happy

* Make tests happy

* Make `LICENSE` symlink

* Make error handling and panic's consistent

* Improve code structure

* Remove unneeded `--workspace`

* Fix `mapping-two-constructor` test file

* Simplify `Option<Vec<_>>` to `Vec`

* Generalize unsetting of env variables in `invoke_cargo`

* Move special case of `dylint` verbosity to call site

* Keep calm and return an `Err`

* Move checking for `dylint-link` to `build.rs`

* Add `ink_linting/Cargo.lock`

* Move `bool_to_exit_code` inline

* Add test for non-fully-qualified path call of `initialize_contract`

* Fix panic when no manifest path was supplied

* Improve structure of `build.rs`

* Make linting error message non-blaming

* Remove incorrect note on not detecting indirections

* Make `clippy` happy

* Impprove text

* Build `ink_linting` with `--locked`

* Fix path separators for Windows CI

* Revert "Fix path separators for Windows CI"

This reverts commit b5998313.

* Revert me: Debug failing Windows CI

* Revert me: Debug failing Windows CI

* Revert me: Debug failing Windows CI

* Revert me: Debug failing Windows CI

* Revert me: Debug failing Windows CI

* Revert me: Debug failing Windows CI

* Revert me: Debug failing Windows CI

* Use `which` for detecting `dylint-link` installation

* Remove `DYLINT_DRIVER_PATH`

* Add `--lib`

* Revert "Remove `DYLINT_DRIVER_PATH`"

This reverts commit d0d209fe

.

* Use temporary `DYLINT_DRIVER_PATH` path during tests

* Give `template` variable a speaking name

Co-authored-by: CI system <>
Co-authored-by: default avatarSergejs Kostjucenko <[email protected]>
parent 30387a52
0% or .
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment