a hidden flag for simpler access to new init format

[balopat]

Fixes #3655.

[codecov]

Codecov Report

Merging #3660 into master will decrease coverage by 0.04%.
The diff coverage is 100%.

Impacted Files	Coverage Δ
pkg/skaffold/initializer/json.go	88.88% <100%> (ø)	⬆️
pkg/skaffold/initializer/init.go	64.81% <100%> (ø)	⬆️
cmd/skaffold/app/cmd/init.go	100% <100%> (ø)	⬆️
...affold/kubernetes/portforward/kubectl_forwarder.go	65.85% <0%> (-2.44%)	⬇️
pkg/skaffold/initializer/analyze.go	87.75% <0%> (-0.49%)	⬇️
pkg/skaffold/build/buildpacks/logger.go	0% <0%> (ø)	⬆️
pkg/skaffold/event/event.go	92.94% <0%> (+0.02%)	⬆️
pkg/skaffold/build/buildpacks/lifecycle.go	75% <0%> (+3.57%)	⬆️

[jonjohnsonjr]

pkg/skaffold/initializer/init.go:54:13: struct of size 400 bytes could be of size 392 bytes (maligned)
type Config struct {

This linter is pedantic! I love it! You can save 4 bytes by moving this up with the other bools.

[balopat]

oh yeah, right, this always gets me

[tejal29]

In future, do you have plans to mark XXenableJibInit and XXenableBuildpacksInit as deprecated and use the new flag instead and convey it to IDEs?

Edit:
Looks like they are used later in detectBuilder for other purpose. There are too many flags now and we should condense them.

[tejal29]

LGTM.

[balopat]

In future, do you have plans to mark XXenableJibInit and XXenableBuildpacksInit as deprecated and use the new flag instead and convey it to IDEs?

Edit:
Looks like they are used later in detectBuilder for other purpose. There are too many flags now and we should condense them.

yeah, they are used separately.
I expect these all to disappear soon!