Skip to content

feat: local integration tests #37

feat: local integration tests

feat: local integration tests #37

GitHub Actions / clippy succeeded May 23, 2024 in 0s

reviewdog [clippy] report

reported by reviewdog 🐶

Findings (0)
Filtered Findings (12)

contracts/src/erc20/mod.rs|3 col 30| warning: item in documentation is missing backticks
--> contracts/src/erc20/mod.rs:3:30
|
3 | //! We have followed general OpenZeppelin Contracts guidelines: functions
| ^^^^^^^^^^^^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#doc_markdown
note: the lint level is defined here
--> contracts/src/lib.rs:3:22
|
3 | #![warn(clippy::all, clippy::pedantic)]
| ^^^^^^^^^^^^^^^^
= note: #[warn(clippy::doc_markdown)] implied by #[warn(clippy::pedantic)]
help: try
|
3 | //! We have followed general OpenZeppelin Contracts guidelines: functions
| ~~~~~~~~~~~~~~
contracts/src/erc721/mod.rs|307 col 15| warning: this argument is passed by value, but not consumed in the function body
--> contracts/src/erc721/mod.rs:307:15
|
307 | data: Bytes,
| ^^^^^ help: consider taking a reference instead: &Bytes
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_value
= note: #[warn(clippy::needless_pass_by_value)] implied by #[warn(clippy::pedantic)]
contracts/src/erc721/mod.rs|473 col 5| warning: this method could have a #[must_use] attribute
--> contracts/src/erc721/mod.rs:473:5
|
473 | / pub fn is_approved_for_all(
474 | | &self,
475 | | owner: Address,
476 | | operator: Address,
477 | | ) -> bool {
| |_^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#must_use_candidate
= note: #[warn(clippy::must_use_candidate)] implied by #[warn(clippy::pedantic)]
help: add the attribute
|
473 ~ #[must_use] pub fn is_approved_for_all(
474 + &self,
475 + owner: Address,
476 + operator: Address,
477 ~ ) -> bool {
|
contracts/src/erc721/mod.rs|497 col 5| warning: this method could have a #[must_use] attribute
--> contracts/src/erc721/mod.rs:497:5
|
497 | pub fn owner_of_inner(&self, token_id: U256) -> Address {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: add the attribute: #[must_use] pub fn _owner_of_inner(&self, token_id: U256) -> Address
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#must_use_candidate
contracts/src/erc721/mod.rs|508 col 5| warning: this method could have a #[must_use] attribute
--> contracts/src/erc721/mod.rs:508:5
|
508 | pub fn get_approved_inner(&self, token_id: U256) -> Address {
| ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: add the attribute: #[must_use] pub fn _get_approved_inner(&self, token_id: U256) -> Address
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#must_use_candidate
contracts/src/erc721/mod.rs|524 col 5| warning: this method could have a #[must_use] attribute
--> contracts/src/erc721/mod.rs:524:5
|
524 | / pub fn is_authorized(
525 | | &self,
526 | | owner: Address,
527 | | spender: Address,
528 | | token_id: U256,
529 | | ) -> bool {
| |
^
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#must_use_candidate
help: add the attribute
|
524 ~ #[must_use] pub fn is_authorized(
525 + &self,
526 + owner: Address,
527 + spender: Address,
528 + token_id: U256,
529 ~ ) -> bool {
|
contracts/src/erc721/mod.rs|737 col 15| warning: this argument is passed by value, but not consumed in the function body
--> contracts/src/erc721/mod.rs:737:15
|
737 | data: Bytes,
| ^^^^^ help: consider taking a reference instead: &Bytes
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_value
contracts/src/erc721/mod.rs|883 col 15| warning: this argument is passed by value, but not consumed in the function body
--> contracts/src/erc721/mod.rs:883:15
|
883 | data: Bytes,
| ^^^^^ help: consider taking a reference instead: &Bytes
|
= help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_pass_by_value
contracts/src/erc721/mod.rs|131 col 1| warning: missing documentation for a struct
--> contracts/src/erc721/mod.rs:131:1
|
131 | / sol_interface! {
132 | | /// ERC-721 token receiver interface.
133 | | ///
134 | | /// Interface for any contract that wants to support safeTransfers
135 | | /// from ERC-721 asset contracts.
136 | | interface IERC721Receiver {
| |_____________________________^
|
note: the lint level is defined here
--> contracts/src/lib.rs:2:9
|
2 | #![warn(missing_docs, unreachable_pub, rust_2021_compatibility)]
| ^^^^^^^^^^^^
= note: this warning originates in the macro sol_interface (in Nightly builds, run with -Z macro-backtrace for more info)
contracts/src/erc721/mod.rs|131 col 1| warning: missing documentation for a struct field
--> contracts/src/erc721/mod.rs:131:1
|
131 | / sol_interface! {
132 | | /// ERC-721 token receiver interface.
133 | | ///
134 | | /// Interface for any contract that wants to support safeTransfers
... |
151 | | }
152 | | }
| |
^
|
= note: this warning originates in the macro sol_interface (in Nightly builds, run with -Z macro-backtrace for more info)
contracts/src/erc721/mod.rs|131 col 1| warning: missing documentation for an associated function
--> contracts/src/erc721/mod.rs:131:1
|
131 | / sol_interface! {
132 | | /// ERC-721 token receiver interface.
133 | | ///
134 | | /// Interface for any contract that wants to support safeTransfers
... |
151 | | }
152 | | }
| |
^
|
= note: this warning originates in the macro sol_interface (in Nightly builds, run with -Z macro-backtrace for more info)
contracts/src/erc721/mod.rs|131 col 1| warning: missing documentation for a method
--> contracts/src/erc721/mod.rs:131:1
|
131 | / sol_interface! {
132 | | /// ERC-721 token receiver interface.
133 | | ///
134 | | /// Interface for any contract that wants to support safeTransfers
... |
151 | | }
152 | | }
| |
^
|
= note: this warning originates in the macro sol_interface (in Nightly builds, run with -Z macro-backtrace for more info)