zarf icon indicating copy to clipboard operation
zarf copied to clipboard

feat: default-required feature flag

Open Noxsios opened this issue 1 year ago • 3 comments

Description

Refer to adr/0025-required-optional.md.

Related Issue

Fixes #2059

Type of change

  • [ ] Bug fix (non-breaking change which fixes an issue)
  • [x] New feature (non-breaking change which adds functionality)
  • [ ] Other (security config, docs update, etc)

Checklist before merging

Noxsios avatar Jan 02 '24 02:01 Noxsios

Deploy Preview for zarf-docs canceled.

Name Link
Latest commit 03689565ec4bbeed90c484b8394457f8be486b0d
Latest deploy log https://app.netlify.com/sites/zarf-docs/deploys/665609da2e32780008319736

netlify[bot] avatar Jan 02 '24 02:01 netlify[bot]

TODO:

  • [x] Talk w/ @Racer159 about composition behavior
  • [x] Discuss zarf-seed-registry import behavior
  • [x] Think about migration behavior
  • [x] Lint addition?
  • [x] zarf package migrate?

Noxsios avatar Jan 02 '24 17:01 Noxsios

Some final thoughts for review:

  • Unit testing filters?
  • config.CommonOptions.Confirm behavior w/ filters.NewDeploymentFilter
  • zarf dev migrate opinionated YAML transformations
  • zarf dev migrate RunE vs Run
  • ADR updates

Noxsios avatar Jan 28 '24 23:01 Noxsios