Skip to content

Commit

Permalink
feat(core): enable the "Override Helm chart artifact" checkbox in man…
Browse files Browse the repository at this point in the history
…ual execution by default (#7818)

Since this is manual execution, it seems like a safe assumption that people want to
specify the helm chart to use.  Leaving it unchecked either leads to pipeline failure, or
to using an artifact from a prior execution (if enabled), both of which are surprising.
  • Loading branch information
dbyron0 authored and mergify[bot] committed Jan 28, 2020
1 parent b51dce4 commit 65b9463
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -9,7 +9,7 @@ import { BAKE_MANIFEST_STAGE_KEY } from './bakeManifestStage';
const HelmEditor = HelmMatch.editCmp;

export function ManualExecutionBakeManifest(props: ITriggerTemplateComponentProps) {
const [overrideArtifact, setOverrideArtifact] = React.useState(false);
const [overrideArtifact, setOverrideArtifact] = React.useState(true);

const updateHelmArtifact = (artifact: IArtifact) => {
const updatedArtifacts = (props.command.extraFields.artifacts || []).filter(
Expand Down

0 comments on commit 65b9463

Please sign in to comment.