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

fix(solc): flatten duplicates #813

Merged
merged 5 commits into from
Jan 19, 2022

Conversation

rkrasiuk
Copy link
Contributor

Motivation

Solution

PR Checklist

  • Added Tests
  • Added Documentation
  • Updated the changelog

ethers-solc/tests/project.rs Outdated Show resolved Hide resolved
Copy link
Owner

@gakonst gakonst left a comment

Choose a reason for hiding this comment

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

CI failure is unrelated lgtm

@gakonst gakonst merged commit f9b0360 into gakonst:master Jan 19, 2022
gakonst added a commit to rkrasiuk/foundry that referenced this pull request Jan 19, 2022
gakonst added a commit to foundry-rs/foundry that referenced this pull request Jan 19, 2022
* add flatten support

* remove linked libs opt

* address PR comments

* chore: bump ethers to fix flatten bug

gakonst/ethers-rs#813

Co-authored-by: Georgios Konstantopoulos <me@gakonst.com>
charisma98 added a commit to charisma98/foundry that referenced this pull request Mar 4, 2023
* add flatten support

* remove linked libs opt

* address PR comments

* chore: bump ethers to fix flatten bug

gakonst/ethers-rs#813

Co-authored-by: Georgios Konstantopoulos <me@gakonst.com>
0129general added a commit to 0129general/FoundryProject that referenced this pull request May 8, 2024
* add flatten support

* remove linked libs opt

* address PR comments

* chore: bump ethers to fix flatten bug

gakonst/ethers-rs#813

Co-authored-by: Georgios Konstantopoulos <me@gakonst.com>
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.

None yet

2 participants