#1733 Update the comments so that use ImageConfig.auto can support all the use case.
Pull request opened by
Future-Outlier
TL;DR
Update the comments so that use ImageConfig.auto can support all the use case.
Type
☑︎ Bug Fix
☐ Feature
☐ Plugin
Are all requirements met?
☐ Code completed
☐ Smoke tested
☐ Unit tests added
☑︎ Code documentation added
☐ Any pending items have an associated Issue
Complete description
I am using the flyte documentation, and I found that flytesnacks doesn't teach how to specify ImageConfig.
After tracing the code in ImageConfig, I found that the old use case (default str) is not a good option.
And I am going to update the flytesnacks !
flyteorg/flytekit
DCO: DCO
✅ 1 other check has passed
1/2 successful checks