refactor(ecs): remove forEach, rename iterate to items

[RigidStudios]

Description

Removes Query#forEach as proposed in #42. Also resolves #47 by renaming Query#iterate to Query#items.

Tasks

• Tests Modified
• Tests Passing

Documentation

• Documentation written
• Documentation submitted
• Documentation published

Linked

Resolves #42
Resolves #47
Resolves AetherInteractiveLtd/docs#2

[RigidStudios]

@christopher-buss Do we need tests for this? There were never any tests for iterate and forEach, I think it's fine, up to you though.

[christopher-buss]

I don't think we should do this. I'd rather have an iterate() and a items() method if you want all the items. We'd have to traverse every entity and add it to a table.. just to re-traverse it later on when we want to use it. I'm happy to have both, but I don't want to remove the generalised iterator completely.

I didn't realise you were doing this, I was currently doing it in another branch!

[RigidStudios]

Alright, I'll restore iterate but rename to iter() then.

[RigidStudios]

I didn't realise you were doing this, I was currently doing it in another branch!

I huffed and puffed about it yesterday, of course I'll do it! 😂

[RigidStudios]

Docs done in AetherInteractiveLtd/docs#2

[RigidStudios]

We'll prefer #55