kukeiko / entity-space

MIT License
5 stars 1 forks source link

merge common, lexer & criteria packages into core package #201

Closed kukeiko closed 1 year ago

kukeiko commented 1 year ago

What

take the following packages:

and move them into the @entity-space/core package

Why

Because as it stands, having those parts as separate packages is unnecessarily overengineered. I first started with the criteria package to be on its own as I thought it would be a good idea for it to be as standalone as possible.

Turns out I then later had to introduce the "common" package as I needed to share EntitySelection logic between core & criteria. I'm sure there will be many more such cases, so I'd rather have everything within one package again.

Additionally, IDE / Jest support is a bit better:

Notes

My current plan for the "end" result is to have the following packages: