Skip to content

Conversation

mablr
Copy link
Contributor

@mablr mablr commented Aug 25, 2025

Motivation

Close #11423

PR Checklist

  • Added Tests
  • Added Documentation
  • Breaking changes

… implementations

This commit simplifies the code by removing the `as_dyn` method from the `MaybeFullDatabase` trait implementations across various database types, as trait upcasting is now stable.
Copy link
Contributor

@0xrusowsky 0xrusowsky left a comment

Choose a reason for hiding this comment

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

lgtm

Copy link
Member

@zerosnacks zerosnacks left a comment

Choose a reason for hiding this comment

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

Thanks! Much better

@DaniPopes DaniPopes merged commit 2c9dbc1 into foundry-rs:master Aug 26, 2025
22 checks passed
@github-project-automation github-project-automation bot moved this to Done in Foundry Aug 26, 2025
MerkleBoy pushed a commit to MerkleBoy/foundry that referenced this pull request Sep 17, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

chore(anvil): get rid of trait upcasting workaround
4 participants