Skip to content

Merge pull request #781 from worldcoin/piohei/add_pre_root_for_identi… #158

Merge pull request #781 from worldcoin/piohei/add_pre_root_for_identi…

Merge pull request #781 from worldcoin/piohei/add_pre_root_for_identi… #158

Triggered via push October 14, 2024 12:51
Status Success
Total duration 13s
Artifacts
update-deployment
4s
update-deployment
Fit to window
Zoom out
Zoom in

Annotations

20 warnings
update-deployment
The following actions use a deprecated Node.js version and will be forced to run on node20: 8BitJonny/[email protected], peter-evans/repository-dispatch@v2. For more info: https://github.blog/changelog/2024-03-07-github-actions-all-actions-will-run-on-node20-instead-of-node16-by-default/
the borrowed expression implements the required traits: src/bin/tool.rs#L221
warning: the borrowed expression implements the required traits --> src/bin/tool.rs:221:23 | 221 | .post(&format!("{}/inclusionProof", args.sequencer_url)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `format!("{}/inclusionProof", args.sequencer_url)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
the borrowed expression implements the required traits: src/bin/tool.rs#L202
warning: the borrowed expression implements the required traits --> src/bin/tool.rs:202:23 | 202 | .post(&format!("{}/verifySemaphoreProof", args.sequencer_url)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `format!("{}/verifySemaphoreProof", args.sequencer_url)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
the borrowed expression implements the required traits: src/bin/tool.rs#L170
warning: the borrowed expression implements the required traits --> src/bin/tool.rs:170:23 | 170 | .post(&format!("{}/inclusionProof", args.sequencer_url)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `format!("{}/inclusionProof", args.sequencer_url)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
the borrowed expression implements the required traits: src/bin/tool.rs#L144
warning: the borrowed expression implements the required traits --> src/bin/tool.rs:144:23 | 144 | .post(&format!("{}/insertIdentity", args.sequencer_url)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `format!("{}/insertIdentity", args.sequencer_url)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args = note: `#[warn(clippy::needless_borrows_for_generic_args)]` on by default
the borrowed expression implements the required traits: src/bin/tool.rs#L221
warning: the borrowed expression implements the required traits --> src/bin/tool.rs:221:23 | 221 | .post(&format!("{}/inclusionProof", args.sequencer_url)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `format!("{}/inclusionProof", args.sequencer_url)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
the borrowed expression implements the required traits: src/bin/tool.rs#L202
warning: the borrowed expression implements the required traits --> src/bin/tool.rs:202:23 | 202 | .post(&format!("{}/verifySemaphoreProof", args.sequencer_url)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `format!("{}/verifySemaphoreProof", args.sequencer_url)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
the borrowed expression implements the required traits: src/bin/tool.rs#L170
warning: the borrowed expression implements the required traits --> src/bin/tool.rs:170:23 | 170 | .post(&format!("{}/inclusionProof", args.sequencer_url)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `format!("{}/inclusionProof", args.sequencer_url)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args
the borrowed expression implements the required traits: src/bin/tool.rs#L144
warning: the borrowed expression implements the required traits --> src/bin/tool.rs:144:23 | 144 | .post(&format!("{}/insertIdentity", args.sequencer_url)) | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ help: change this to: `format!("{}/insertIdentity", args.sequencer_url)` | = help: for further information visit https://rust-lang.github.io/rust-clippy/master/index.html#needless_borrows_for_generic_args = note: `#[warn(clippy::needless_borrows_for_generic_args)]` on by default
field `legacy` is never read: src/ethereum/read/mod.rs#L21
warning: field `legacy` is never read --> src/ethereum/read/mod.rs:21:9 | 18 | pub struct ReadProvider { | ------------ field in this struct ... 21 | pub legacy: bool, | ^^^^^^ | = note: `ReadProvider` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis
fields `batch_next_root` and `transaction_id` are never read: src/database/types.rs#L89
warning: fields `batch_next_root` and `transaction_id` are never read --> src/database/types.rs:89:9 | 88 | pub struct TransactionEntry { | ---------------- fields in this struct 89 | pub batch_next_root: Hash, | ^^^^^^^^^^^^^^^ 90 | pub transaction_id: String, | ^^^^^^^^^^^^^^ | = note: `TransactionEntry` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
methods `get_latest_root`, `get_latest_batch_with_transaction`, `get_all_batches_after`, `root_in_batch_chain`, and `get_transaction_for_batch` are never used: src/database/query.rs#L189
warning: methods `get_latest_root`, `get_latest_batch_with_transaction`, `get_all_batches_after`, `root_in_batch_chain`, and `get_transaction_for_batch` are never used --> src/database/query.rs:189:14 | 24 | pub trait DatabaseQuery<'a>: Executor<'a, Database = Postgres> { | ------------- methods in this trait ... 189 | async fn get_latest_root(self) -> Result<Option<Hash>, Error> { | ^^^^^^^^^^^^^^^ ... 704 | async fn get_latest_batch_with_transaction(self) -> Result<Option<BatchEntry>, Error> { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ ... 770 | async fn get_all_batches_after(self, id: i64) -> Result<Vec<BatchEntry>, Error> { | ^^^^^^^^^^^^^^^^^^^^^ ... 816 | async fn root_in_batch_chain(self, root: &Hash) -> Result<bool, Error> { | ^^^^^^^^^^^^^^^^^^^ ... 846 | async fn get_transaction_for_batch( | ^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: `#[warn(dead_code)]` on by default
field `legacy` is never read: src/ethereum/read/mod.rs#L21
warning: field `legacy` is never read --> src/ethereum/read/mod.rs:21:9 | 18 | pub struct ReadProvider { | ------------ field in this struct ... 21 | pub legacy: bool, | ^^^^^^ | = note: `ReadProvider` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis
fields `batch_next_root` and `transaction_id` are never read: src/database/types.rs#L89
warning: fields `batch_next_root` and `transaction_id` are never read --> src/database/types.rs:89:9 | 88 | pub struct TransactionEntry { | ---------------- fields in this struct 89 | pub batch_next_root: Hash, | ^^^^^^^^^^^^^^^ 90 | pub transaction_id: String, | ^^^^^^^^^^^^^^ | = note: `TransactionEntry` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `existing_commitment` is never read: src/database/types.rs#L23
warning: field `existing_commitment` is never read --> src/database/types.rs:23:9 | 22 | pub struct RecoveryEntry { | ------------- field in this struct 23 | pub existing_commitment: Hash, | ^^^^^^^^^^^^^^^^^^^
multiple methods are never used: src/database/query.rs#L189
warning: multiple methods are never used --> src/database/query.rs:189:14 | 24 | pub trait DatabaseQuery<'a>: Executor<'a, Database = Postgres> { | ------------- methods in this trait ... 189 | async fn get_latest_root(self) -> Result<Option<Hash>, Error> { | ^^^^^^^^^^^^^^^ ... 442 | async fn get_all_recoveries(self) -> Result<Vec<RecoveryEntry>, Error> { | ^^^^^^^^^^^^^^^^^^ ... 662 | async fn get_next_batch(self, prev_root: &Hash) -> Result<Option<BatchEntry>, Error> { | ^^^^^^^^^^^^^^ ... 704 | async fn get_latest_batch_with_transaction(self) -> Result<Option<BatchEntry>, Error> { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ ... 770 | async fn get_all_batches_after(self, id: i64) -> Result<Vec<BatchEntry>, Error> { | ^^^^^^^^^^^^^^^^^^^^^ ... 816 | async fn root_in_batch_chain(self, root: &Hash) -> Result<bool, Error> { | ^^^^^^^^^^^^^^^^^^^ ... 846 | async fn get_transaction_for_batch( | ^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: `#[warn(dead_code)]` on by default
field `legacy` is never read: src/ethereum/read/mod.rs#L21
warning: field `legacy` is never read --> src/ethereum/read/mod.rs:21:9 | 18 | pub struct ReadProvider { | ------------ field in this struct ... 21 | pub legacy: bool, | ^^^^^^ | = note: `ReadProvider` has derived impls for the traits `Debug` and `Clone`, but these are intentionally ignored during dead code analysis
fields `batch_next_root` and `transaction_id` are never read: src/database/types.rs#L89
warning: fields `batch_next_root` and `transaction_id` are never read --> src/database/types.rs:89:9 | 88 | pub struct TransactionEntry { | ---------------- fields in this struct 89 | pub batch_next_root: Hash, | ^^^^^^^^^^^^^^^ 90 | pub transaction_id: String, | ^^^^^^^^^^^^^^ | = note: `TransactionEntry` has derived impls for the traits `Clone` and `Debug`, but these are intentionally ignored during dead code analysis
field `existing_commitment` is never read: src/database/types.rs#L23
warning: field `existing_commitment` is never read --> src/database/types.rs:23:9 | 22 | pub struct RecoveryEntry { | ------------- field in this struct 23 | pub existing_commitment: Hash, | ^^^^^^^^^^^^^^^^^^^
multiple methods are never used: src/database/query.rs#L189
warning: multiple methods are never used --> src/database/query.rs:189:14 | 24 | pub trait DatabaseQuery<'a>: Executor<'a, Database = Postgres> { | ------------- methods in this trait ... 189 | async fn get_latest_root(self) -> Result<Option<Hash>, Error> { | ^^^^^^^^^^^^^^^ ... 442 | async fn get_all_recoveries(self) -> Result<Vec<RecoveryEntry>, Error> { | ^^^^^^^^^^^^^^^^^^ ... 662 | async fn get_next_batch(self, prev_root: &Hash) -> Result<Option<BatchEntry>, Error> { | ^^^^^^^^^^^^^^ ... 704 | async fn get_latest_batch_with_transaction(self) -> Result<Option<BatchEntry>, Error> { | ^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^ ... 770 | async fn get_all_batches_after(self, id: i64) -> Result<Vec<BatchEntry>, Error> { | ^^^^^^^^^^^^^^^^^^^^^ ... 816 | async fn root_in_batch_chain(self, root: &Hash) -> Result<bool, Error> { | ^^^^^^^^^^^^^^^^^^^ ... 846 | async fn get_transaction_for_batch( | ^^^^^^^^^^^^^^^^^^^^^^^^^ | = note: `#[warn(dead_code)]` on by default