search

0xPolygonZero / zk_evm

Apache License 2.0
85 stars 37 forks source link

feat: add ci cargo outdated check #752

Closed atanmarko closed 2 weeks ago

atanmarko commented 2 weeks ago

Resolves https://github.com/0xPolygonZero/zk_evm/issues/746

Note: cargo-outdated does not error. It produces list of current, compatible and latest versions of the dependencies currently used. This CI job saves the dependency list table as an artifact.

atanmarko commented 2 weeks ago

@0xaatif Problem here is to know when to generate warning and not false warning - cargo outdated merely lists the versions, it seems to me that it doesn't claim something is outdated.

0xaatif commented 2 weeks ago

How do you feel about the approach in https://github.com/0xPolygonZero/zk_evm/pull/757 instead?

atanmarko commented 2 weeks ago

We will use alternative implementation in #757