-
Notifications
You must be signed in to change notification settings - Fork 344
Add everpuzzle to newsletter 7 #69
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
Conversation
Thanks for this, I approved it but I think the sentence needs some tweaking:
|
sure, i somehow have to explain the move away from amethyst in the nicest way, any suggestion? |
@Skytrias I think it's important to be critical of Amethyst if you have reason to be. You used it, and have knowledge of ways it didn't work. However, saying it bluntly doesn't really fit the newsletter I think. It would do better as a part of a devblog post instead. Maybe it could say this instead?
|
Saying that you changed from amethyst to your own ECS system is probably fine. E.g.
|
thing is im just using plain enums now instead of ecs, so is that really my own entity management? |
that sounds better! |
Fixed a few markdown issues and resolved merge conflicts. |
⏲️ Merging in a few hours if there're no objections. |
Merged! Thanks for the PR! |
Tried to keep it smaller
Part of #64 ("Newsletter 7: Coordination/Tracking")