-
Notifications
You must be signed in to change notification settings - Fork 5.5k
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
Add EIP: Entity Component System #7651
Conversation
File
|
In the future, please don't create a new pull request. Just push to the same one until it's approved. Thanks! |
Hey @SamWilsn Please check again. Do I need to modify anything else? thank you. |
Hey @axic @Pandapip1 @SamWilsn @xinbenlv |
this is fairly interesting, would give it an indepth look :) |
Thanks @g11tech , I'm working on some tools for it, FYI |
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.
looks good w.r.t. enshrining ECS interface 👍
I am closing this pull request because we are in the process of separating EIPs and ERCs into distinct repositories. Unfortunately, as far as we are aware, GitHub does not provide any tools to ease this migration, so every pull request will need to be re-opened manually. As this is a PR to create / modify an ERC, I will kindly ask you to redirect this to the new repository at ethereum/ERCs. We have prepared a guide to help with the process. If there is relevant history here, please link to this PR from the new pull request. On behalf of the EIP Editors, I apologize for this inconvenience. |
@HelloRickey Sorry it's taken so long to get a review on this! I was caught up with reviewing EIPs submitted before sometime in June before we split the repositories... Once this is re-opened on the new repository, I'll give it a look when I can! |
Discussions on:https://ethereum-magicians.org/t/a-new-proposal-of-entity-component-system/15665
Fix issue raised by @SamWilsn
#7641