Namespace and type naming collisions about orion-core HOT 6 CLOSED

• Orion.* -> OrionApi.*
• Orion.Entities.<name> -> OrionApi.Entities.<name>s
• Orion.Events.<name> -> OrionApi.Events.<name>s

EDIT: where <name> != World

from orion-core.

I think renaming the Orion namespace to something like OrionApi is fitting and acceptable.

What about prefixing the entity names? Something like OrionItem, OrionNpc, etc. That way we have differentiation in the code as from the Terraria entities. (As well as name differentiation).

We can probably then leave the namespaces the same.

from orion-core.

I do like prefixing the entity names, but you would still have a conflict with the namespace and the Terraria entity. Since the concrete types are going into TShock, we would name them TshockItem, TshockNpc, etc.

Also, something to note is that our type names conflicting with Terraria names shouldn't make too much of a difference in our name decisions, since our goal is to eventually have consumers not need to reference anything in the Terraria namespace.

from orion-core.

Prefixing them with TShock (or Tshock, whatever) is good. I would agree with pluralizing the namespaces.

Note: Ignore accidental comment+close

from orion-core.

If there are no objections to the namespace renaming, then I will do this tonight.

from orion-core.

Please detail the exact changes you'll make, for the sake of having it on record+everyone being on the same page.

from orion-core.