Skip to content

feat(evm): dependencies#267

Merged
hedgar2017 merged 1 commit intoaz-several-qol-fixesfrom
az-support-evm-dependencies
Mar 19, 2025
Merged

feat(evm): dependencies#267
hedgar2017 merged 1 commit intoaz-several-qol-fixesfrom
az-support-evm-dependencies

Conversation

@hedgar2017
Copy link
Copy Markdown
Contributor

Supports CREATE-dependencies in the EVM target.

Checklist

  • PR title corresponds to the body of PR.
  • 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.

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 13, 2025

Codecov Report

Attention: Patch coverage is 88.88889% with 33 lines in your changes missing coverage. Please review.

Project coverage is 73.21%. Comparing base (853b584) to head (f66af3f).

Files with missing lines Patch % Lines
era-compiler-solidity/src/project/contract/mod.rs 73.21% 15 Missing ⚠️
era-yul/src/yul/dependencies.rs 63.33% 11 Missing ⚠️
era-yul/src/yul/parser/statement/mod.rs 75.00% 4 Missing ⚠️
era-compiler-solidity/src/evmla/assembly/data.rs 66.66% 2 Missing ⚠️
era-compiler-solidity/src/evmla/assembly/mod.rs 93.75% 1 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #267      +/-   ##
==========================================
+ Coverage   73.04%   73.21%   +0.16%     
==========================================
  Files         132      135       +3     
  Lines       15576    15806     +230     
==========================================
+ Hits        11378    11572     +194     
- Misses       4198     4234      +36     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@hedgar2017 hedgar2017 force-pushed the az-support-evm-dependencies branch 2 times, most recently from 44b60ae to 97f3ff9 Compare February 18, 2025 11:10
@hedgar2017 hedgar2017 force-pushed the az-support-evm-dependencies branch 7 times, most recently from 763f86b to b26b51c Compare March 12, 2025 04:04
@hedgar2017 hedgar2017 marked this pull request as ready for review March 18, 2025 18:07
@hedgar2017 hedgar2017 requested a review from a team March 18, 2025 18:23
Copy link
Copy Markdown
Contributor

@PavelKopyl PavelKopyl left a comment

Choose a reason for hiding this comment

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

LGTM

@hedgar2017 hedgar2017 changed the base branch from main to az-several-qol-fixes March 19, 2025 06:55
@hedgar2017 hedgar2017 force-pushed the az-support-evm-dependencies branch from a2cbcb3 to c9a4b95 Compare March 19, 2025 07:03
@hedgar2017 hedgar2017 force-pushed the az-support-evm-dependencies branch from c9a4b95 to 4082400 Compare March 19, 2025 07:06
@hedgar2017 hedgar2017 merged commit 4244bf1 into az-several-qol-fixes Mar 19, 2025
5 of 12 checks passed
@hedgar2017 hedgar2017 deleted the az-support-evm-dependencies branch March 19, 2025 07:09
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