chore: typos in scroll-tech / zkevm-circuits

[alexanderblv]

zkevm-circuits/zktrie/src/state.rs

Line 72 in a08da07

/// new snapshot since we consider it is not need to send more nodes data

"is not need" should be "is not needed"

zkevm-circuits/bus-mapping/src/rpc.rs

Line 118 in a08da07

/// returning all the block information including it's transaction's

"it's" should be "its"

zkevm-circuits/gadgets/src/is_zero.rs

Line 4 in a08da07

//! - witnesses `inv0(value)`, where `inv0(x)` is 0 when `x` = 0, and

"witnesses" should be "witnesses:"

zkevm-circuits/gadgets/src/is_zero.rs

Line 17 in a08da07

/// implement `IsZero`.

"implementation" should be "implementing"

[chenyuan1189]

no

[alexanderblv]

no

I don't quite understand what "No" means.

[Kajko25]

no

I don't quite understand what "No" means.

Neither do I... unfortunately it seems to me that this user had no intention of contributing anything to the discussion.... maybe someone from the team will speak up?

[alexanderblv]

no

I don't quite understand what "No" means.

Neither do I... unfortunately it seems to me that this user had no intention of contributing anything to the discussion.... maybe someone from the team will speak up?

It would be good. Unfortunately, the issue has been open for a long time, but there is no reaction from the team. Maybe @DreamWuGit will respond

[lispc]

thanks for the typos pointed. We may need to add some spelling check in CI to fix these and prevent typos in future. TODO.

[drstevenbrule]

Disagree with your last one but made a PR for the sake of closing this out.

[xyq-c-cpp]

thanks for the typos pointed. We may need to add some spelling check in CI to fix these and prevent typos in future. TODO.

can I do this TODO ? I would deliver it as soon as possible. thanks a lot. @lispc

[lispc]

thanks for the typos pointed. We may need to add some spelling check in CI to fix these and prevent typos in future. TODO.

can I do this TODO ? I would deliver it as soon as possible. thanks a lot. @lispc

that's welcome!