Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat(evm): EVMLA (E, I) pipelines #120

Merged
merged 3 commits into from
Nov 22, 2024

Conversation

hedgar2017
Copy link
Collaborator

What ❔

Fixes the EVMLA pipeline for EVM target.

Why ❔

Some deploy code functions were unsupported.

Checklist

  • PR title corresponds to the body of PR (we generate changelog entries from PRs).
  • Tests for the changes have been added / updated.
  • Documentation comments have been added / updated.
  • Code has been formatted via cargo fmt and checked with cargo clippy.

Copy link
Contributor

@antonbaliasnikov antonbaliasnikov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

By some reason licensing started to fail:

error[rejected]: failed to satisfy license requirements
  ┌─ registry+https://github.com/rust-lang/crates.io-index#[email protected]:4:12
  │
4 │ license = "Unicode-3.0"
  │            ^^^^^^^^^^^
  │            │
  │            license expression retrieved via Cargo.toml `license`
  │            rejected: not explicitly allowed

@hedgar2017
Copy link
Collaborator Author

@antonbaliasnikov yeah, that seems to have been triggered by some external intervention.

@hedgar2017 hedgar2017 force-pushed the az-cpr-1829-unblock-the-evmla-pipeline-for-evm branch from e2142af to 4cdad2a Compare November 21, 2024 09:12
@hedgar2017
Copy link
Collaborator Author

To be merged after matter-labs/era-compiler-vyper#127 where main still depends on structopt.

@antonbaliasnikov
Copy link
Contributor

@hedgar2017 , I've updated cargo.lock and restarted CI.

@hedgar2017 hedgar2017 force-pushed the az-cpr-1829-unblock-the-evmla-pipeline-for-evm branch from 74cda40 to 5fc6a8b Compare November 22, 2024 14:54
@hedgar2017
Copy link
Collaborator Author

@antonbaliasnikov it looks like the checks are still not mandatory here.
Could you please check if it's so in all repos?

@hedgar2017 hedgar2017 merged commit a0c56d9 into main Nov 22, 2024
4 of 6 checks passed
@hedgar2017 hedgar2017 deleted the az-cpr-1829-unblock-the-evmla-pipeline-for-evm branch November 22, 2024 14:55
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants