[Draft] Implement radar code as a separate component - Githubissues

vegastrike / Vega-Strike-Engine-Source

Vega Strike Engine

Other

256 stars 42 forks source link

[Draft] Implement radar code as a separate component #804

Open royfalk opened 11 months ago

royfalk commented 11 months ago

This PR introduces the concept of components - reactor, radar, cloaking device, etc. This allows us to more elegantly move code from unit to components. It will also allow for the removal of unit superclasses (e.g. movable) and easier upgrade, buy, sell, damage and fix components.

Let's use this to PR (and the cloaking one) discuss the concept of using components as a way forward.

Please answer the following:

Code Changes:

[not yet] Have the PR Validation Tests been run? See https://github.com/vegastrike/Vega-Strike-Engine-Source/wiki/Pull-Request-Validation

Issues:

None yet

Purpose:

What is this pull request trying to do?
What release is this for?
Is there a project or milestone we should apply this to?

BenjamenMeyer commented 11 months ago

Yes, I like moving things to components. It makes a lot of sense.