Lookup mechanism in `stacker.lookups` is unused but still present and tested

cloudtools / stacker

An AWS CloudFormation Stack orchestrator/manager.

http://stacker.readthedocs.io/en/stable/

BSD 2-Clause "Simplified" License

709 stars 167 forks source link

Lookup mechanism in `stacker.lookups` is unused but still present and tested #718

Open danielkza opened 5 years ago

danielkza commented 5 years ago

It seems the lookup handling logic has been rewritten and is now almost completely contained in stacker.variables.Variable, but the code in stacker.lookups is still included and test.

We should remove that code, and if keeping compatibility is needed, make them share the same code.