Issues · garden-io/garden
# 💻|contributing
s
While looking into [this issue](https://github.com/garden-io/garden/issues/5317), I noticed that the logic for parsing Dockerfiles and computing includes from them is only being used for module configs (not action configs). I.e. the
containerHelpers.autoResolveIncludes
helper. Was this done on purpose (i.e. did we decide to deprecate the automatic inference of includes from Dockerfiles but still support it for current module configs), or should I fix that while I'm at it? @alert-helicopter-61082 Also CC-ing @big-spring-14945 since he recently did some fixes in this part of the codebase.
b
I wasn't aware of that and I see no reason not to automatically resolve includes for Dockerfile build action
@User So, feel free to go ahead and fix this 👍
c
@swift-garage-61180 are you planning to work on this? I can take something else if you are.
s
@curved-intern-91221 Go ahead! 👍 I was going to, but I'm going to be busy with other things for the next few days, so It would be great if you took this on.
c
Sure, thanks! I'll take a look tomorrow. Fixed some sync mode issues today)