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

RT ECS docs rewrite #343

Open
wants to merge 10 commits into
base: master
Choose a base branch
from
Open

RT ECS docs rewrite #343

wants to merge 10 commits into from

Conversation

benev0
Copy link

@benev0 benev0 commented Nov 11, 2024

This rewrite of the ECS doc has the goal of simplifying explanations of the architecture pattern.

  • split the document into atomic parts
  • remove and replace composition over inheritance diatribe (needed for DM programmers)
  • provide minor edits to current quality content (event patterns)

fix #15

@benev0 benev0 marked this pull request as ready for review November 17, 2024 00:51
@benev0
Copy link
Author

benev0 commented Nov 19, 2024

I need to revisit FAQ.

@benev0
Copy link
Author

benev0 commented Nov 21, 2024

The system example does not have a update system in it.

@benev0
Copy link
Author

benev0 commented Dec 2, 2024

Added update to code example, but it may be worth splitting the sample up so that it is present adjacent to each of the appropriate sections. seeking opinions.

I would appreciate a read through by someone who is not me. If you find something that is difficult to read, verbose, incorrect, or flawed, then feel free to mark it with anything from "here" or "this" to a detailed explanation of an issue. The latter is preferred for incorrect technical information.

If you do not understand ECS concepts before reading and do not understand during or after reading, your review is very valuable to me, so please post your review. The goal is to bring you up to speed on ECS; you are the target audience.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

ECS doc is bad and also severely out of date
1 participant