-
Notifications
You must be signed in to change notification settings - Fork 12
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do you think the folder structure under test/foundry
should be consistent as under the contracts
?
Also, I think there are quite a lot of files under the inputs
folder, which makes it difficult to figure out where to start reading. Do you think we should create new folders under inputs
for those renamed files? We could consider a structure like:
- inputs
- input relay
- address relay
- portals
@ZzzzHui After some thought, I think I agree with you on this. I will revert the moving of contracts. :-) |
2ea6729
to
727782c
Compare
Rebased and reverted moving of contracts. :-) |
727782c
to
7082820
Compare
Fixed some paths. :-) |
👍 Just need to rebase the branch |
7082820
to
0824521
Compare
Rebased 🙌 |
This PR is ready to merge |
0824521
to
682bd36
Compare
Rebased, and ran |
* Make portals and relays inherit `InputRelay` * Make portals and relays interfaces inherit `IInputRelay`
682bd36
to
b747ccc
Compare
Rebased. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM!
Proposed changes
contracts/inputs
calledInputRelay
InputRelay
Portal
andRelay
Related issues
Closes #131.
Footnotes
and associated interface(s) ↩ ↩2 ↩3