Skip to content

Add no_main lint #2001

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
jubnzv merged 11 commits into from
Nov 28, 2023
Merged

Add no_main lint #2001

Changes from 1 commit
Commits
Show all changes
11 commits
Select commit Hold shift + click to select a range
e5635b9
feat(linter): Add `no_main` lint
jubnzv Nov 11, 2023
aebee32
feat(lint): Run lint only for onchain compilation targets
jubnzv Nov 17, 2023
4033d24
chore(lint): Linting message
jubnzv Nov 17, 2023
34f236d
chore(lint): Remove the `fail` test
jubnzv Nov 17, 2023
3a4955c
chore: Add CHANGELOG entry
jubnzv Nov 20, 2023
05b3a42
fix: Remove fail test from Cargo.toml
jubnzv Nov 20, 2023
c977fa6
fix(lint): Comments, lint message
jubnzv Nov 21, 2023
bbec165
Merge remote-tracking branch 'origin/master' into 1976-no-main
jubnzv Nov 28, 2023
a1f87d9
fix(lint): Run `no_main` on e2e tests
jubnzv Nov 28, 2023
7b237d3
fix: fmt
jubnzv Nov 28, 2023
3e17b24
fix: bless tests after fmt fix
jubnzv Nov 28, 2023
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
fix: bless tests after fmt fix
  • Loading branch information
@jubnzv
jubnzv committed Nov 28, 2023
commit 3e17b24035dd6342e3669982a72f2f3293f9c594
16 changes: 8 additions & 8 deletions linting/ui/fail/storage_never_freed.stderr
View file
Open in desktop
Original file line number Diff line number Diff line change
@@ -1,58 +1,58 @@
error: field's storage cannot be freed
--> $DIR/storage_never_freed.rs:19:9
--> $DIR/storage_never_freed.rs:18:9
|
LL | map_1: Mapping<AccountId, AccountId>,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: consider adding operations to remove elements available to the user
note: the lint level is defined here
--> $DIR/storage_never_freed.rs:3:46
--> $DIR/storage_never_freed.rs:2:46
|
LL | #![cfg_attr(dylint_lib = "ink_linting", deny(storage_never_freed))]
| ^^^^^^^^^^^^^^^^^^^

error: field's storage cannot be freed
--> $DIR/storage_never_freed.rs:20:9
--> $DIR/storage_never_freed.rs:19:9
|
LL | map_2: Mapping<AccountId, AccountId>,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: consider adding operations to remove elements available to the user

error: field's storage cannot be freed
--> $DIR/storage_never_freed.rs:21:9
--> $DIR/storage_never_freed.rs:20:9
|
LL | map_3: Mapping<AccountId, AccountId>,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: consider adding operations to remove elements available to the user

error: field's storage cannot be freed
--> $DIR/storage_never_freed.rs:22:9
--> $DIR/storage_never_freed.rs:21:9
|
LL | map_alias: MapAlias2<AccountId, AccountId>,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
|
= help: consider adding operations to remove elements available to the user

error: field's storage cannot be freed
--> $DIR/storage_never_freed.rs:16:9
--> $DIR/storage_never_freed.rs:15:9
|
LL | vec_1: Vec<AccountId>,
| ^^^^^^^^^^^^^^^^^^^^^
|
= help: consider adding operations to remove elements available to the user

error: field's storage cannot be freed
--> $DIR/storage_never_freed.rs:17:9
--> $DIR/storage_never_freed.rs:16:9
|
LL | vec_2: Vec<bool>,
| ^^^^^^^^^^^^^^^^
|
= help: consider adding operations to remove elements available to the user

error: field's storage cannot be freed
--> $DIR/storage_never_freed.rs:18:9
--> $DIR/storage_never_freed.rs:17:9
|
LL | vec_subscription: Vec<AccountId>,
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
Expand Down