You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
privatefinalContainerBuildPlan.BuildercontainerBuildPlanBuilder = ContainerBuildPlan.builder();
// TODO(chanseok): remove and use containerBuildPlanBuilder instead. Note that// ContainerConfiguation implements equals() and hashCode(), so need to verify// if they are required.privatefinalContainerConfiguration.BuildercontainerConfigurationBuilder =
ContainerConfiguration.builder();
The two classes are basically identical. As of now, JibContainerBuilder is using both (i.e., duplicating configuration in each class): ContainerConfiguration for internal and historical use with the jib-core backend code, and ContainerBuildPlan for public API.
layerConfigurations can be removed too.
// TODO(chanseok): remove and use containerBuildPlanBuilder instead.
private List<FileEntriesLayer> layerConfigurations = new ArrayList<>();
The text was updated successfully, but these errors were encountered:
chanseokoh
changed the title
Replace ContainerConfiguration with ContainerBuildPlan
Substitute ContainerBuildPlan for ContainerConfiguration and JibContainerBuilder.layerConfigurations
Jul 24, 2020
As mentioned in JibContainerBuilder.
The two classes are basically identical. As of now,
JibContainerBuilder
is using both (i.e., duplicating configuration in each class):ContainerConfiguration
for internal and historical use with the jib-core backend code, andContainerBuildPlan
for public API.layerConfigurations
can be removed too.The text was updated successfully, but these errors were encountered: