icenet-ai / icenet-etl

Infrastructure for storing IceNet predictions and importing them into a database
MIT License
1 stars 0 forks source link

Review resource naming #47

Open JimCircadian opened 10 months ago

JimCircadian commented 10 months ago

Noticed in the forecast-processor module that the naming is really inconsistent, with resources being suffixed either fcproc, evtproc or event-processor.

A good first issue would be to go through and ensure that the module sets the suffix consistently with a local variable: eg prefix-${var.project_name}-${local.app_name} or something like that. This would allow someone to have a good nosey through all the resources to get a feel for the code.