starkware-libs / cairo

Cairo is the first Turing-complete language for creating provable programs for general computation.
Apache License 2.0
1.56k stars 479 forks source link

Valid compiler version default value in CasmContractClass #6381

Closed TzahiTaub closed 6 days ago

TzahiTaub commented 2 weeks ago

This change is Reviewable

ilyalesokhin-starkware commented 2 weeks ago

crates/cairo-lang-starknet-classes/src/casm_contract_class.rs line 110 at r1 (raw file):


/// Represents a contract in the Starknet network.
#[derive(Clone, Default, Debug, PartialEq, Eq, Serialize, Deserialize)]

I think we should just remove the default here.

Code quote:

Default,
TzahiTaub commented 6 days ago

Decided to remove the default impl entirely.