Closed InsertCreativityHere closed 6 months ago
This PR merges two of the SliceConfig fields into one:
SliceConfig
include_built_in_path: bool, built_in_slice_path: String,
become:
built_in_slice_path: Option<String>,
The bool becomes the Option (true = Some, false = None) And the option holds the actual path inside itself.
Option
This prevents the "Weird case where include_built_in_path is true but built_in_slice_path is empty."
include_built_in_path
built_in_slice_path
No logic is changed by this PR. It should be strictly equivalent to the current logic.
This PR merges two of the
SliceConfigfields into one:become:
The bool becomes the
Option(true = Some, false = None) And the option holds the actual path inside itself.This prevents the "Weird case where
include_built_in_pathis true butbuilt_in_slice_pathis empty."No logic is changed by this PR. It should be strictly equivalent to the current logic.