Skip to content

Add ink! linting MVP stage #431

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.

Sign up for GitHub

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

Merged
cmichi merged 45 commits into from
Feb 23, 2022
Merged

Add ink! linting MVP stage #431

Changes from 1 commit
Commits
Show all changes
45 commits
Select commit Hold shift + click to select a range
82a8d4c
Add ink! linting MVP stage
cmichi Feb 11, 2022
36c5c99
CI: cache cargo installations; update versions
Feb 17, 2022
203ae16
Apply suggestions from code review
cmichi Feb 18, 2022
565c433
Merge branch 'master' into cmichi-implemnt-dylint-mvp
cmichi Feb 18, 2022
618ea70
Update changelog
cmichi Feb 18, 2022
0536795
Check if `dylint` requirements are installed
cmichi Feb 18, 2022
2e18422
Merge branch 'master' into cmichi-implemnt-dylint-mvp
cmichi Feb 18, 2022
5035790
Make `clippy` happy
cmichi Feb 18, 2022
35107a8
Make tests happy
cmichi Feb 18, 2022
36e85d1
Make `LICENSE` symlink
cmichi Feb 22, 2022
2fc61c1
Make error handling and panic's consistent
cmichi Feb 22, 2022
26edca5
Improve code structure
cmichi Feb 22, 2022
26bf192
Remove unneeded `--workspace`
cmichi Feb 22, 2022
23af5eb
Fix `mapping-two-constructor` test file
cmichi Feb 22, 2022
aeb5287
Simplify `Option<Vec<_>>` to `Vec`
cmichi Feb 22, 2022
44fbfb9
Generalize unsetting of env variables in `invoke_cargo`
cmichi Feb 22, 2022
424a1e9
Move special case of `dylint` verbosity to call site
cmichi Feb 22, 2022
f8f9b8f
Keep calm and return an `Err`
cmichi Feb 22, 2022
db67998
Move checking for `dylint-link` to `build.rs`
cmichi Feb 23, 2022
2065d20
Add `ink_linting/Cargo.lock`
cmichi Feb 23, 2022
8367c5b
Move `bool_to_exit_code` inline
cmichi Feb 23, 2022
402a28d
Add test for non-fully-qualified path call of `initialize_contract`
cmichi Feb 23, 2022
60287bd
Fix panic when no manifest path was supplied
cmichi Feb 23, 2022
d0071ee
Improve structure of `build.rs`
cmichi Feb 23, 2022
87289ed
Make linting error message non-blaming
cmichi Feb 23, 2022
1fc817c
Remove incorrect note on not detecting indirections
cmichi Feb 23, 2022
16d77c6
Make `clippy` happy
cmichi Feb 23, 2022
98d2543
Impprove text
cmichi Feb 23, 2022
b1be397
Build `ink_linting` with `--locked`
cmichi Feb 23, 2022
b599831
Fix path separators for Windows CI
cmichi Feb 23, 2022
892f459
Revert "Fix path separators for Windows CI"
cmichi Feb 23, 2022
03a5fba
Revert me: Debug failing Windows CI
cmichi Feb 23, 2022
78db3f5
Revert me: Debug failing Windows CI
cmichi Feb 23, 2022
7a26bc9
Revert me: Debug failing Windows CI
cmichi Feb 23, 2022
cea5ab5
Revert me: Debug failing Windows CI
cmichi Feb 23, 2022
14af20e
Revert me: Debug failing Windows CI
cmichi Feb 23, 2022
f60e682
Revert me: Debug failing Windows CI
cmichi Feb 23, 2022
29b27a2
Revert me: Debug failing Windows CI
cmichi Feb 23, 2022
81bde92
Use `which` for detecting `dylint-link` installation
cmichi Feb 23, 2022
68a1562
Merge branch 'master' into cmichi-implemnt-dylint-mvp
cmichi Feb 23, 2022
d0d209f
Remove `DYLINT_DRIVER_PATH`
cmichi Feb 23, 2022
0ea9417
Add `--lib`
cmichi Feb 23, 2022
a00c562
Revert "Remove `DYLINT_DRIVER_PATH`"
cmichi Feb 23, 2022
959eb44
Use temporary `DYLINT_DRIVER_PATH` path during tests
cmichi Feb 23, 2022
d417899
Give `template` variable a speaking name
cmichi Feb 23, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
Prev Previous commit
Next Next commit
Merge branch 'master' into cmichi-implemnt-dylint-mvp
  • Loading branch information
@cmichi
cmichi committed Feb 23, 2022
commit 68a1562ccbf0808bfcffc620fea327eee15462f9
24 changes: 12 additions & 12 deletions Cargo.lock
View file
Open in desktop

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

You are viewing a condensed version of this merge commit. You can view the full changes here.