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

Ensure traits from other modules can be implemented #773

Merged
merged 1 commit into from
Aug 3, 2022

Conversation

cburgdorf
Copy link
Collaborator

@cburgdorf cburgdorf commented Jul 29, 2022

What was wrong?

It is not possible to implement a trait that is defined in another ingot or even just another module.

How was it fixed?

Do not just look for implementations in the module of the trait but everywhere.

@cburgdorf cburgdorf force-pushed the christoph/fix/foreign_traits branch from c496f1d to 06e632b Compare July 29, 2022 09:40
@cburgdorf cburgdorf requested a review from sbillig July 29, 2022 09:42
@cburgdorf cburgdorf force-pushed the christoph/fix/foreign_traits branch from 06e632b to 58eb381 Compare July 29, 2022 11:34
Copy link
Member

@Y-Nak Y-Nak left a comment

Choose a reason for hiding this comment

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

👍

@Y-Nak Y-Nak merged commit 51787e9 into ethereum:master Aug 3, 2022
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