Skip to content

Commit e5814ba

Browse files
committed
update comments / doc since crdOptions and upstreamOptions are no longer mutually exclusive
1 parent fba2a7d commit e5814ba

File tree

2 files changed

+2
-4
lines changed

2 files changed

+2
-4
lines changed

pkg/options/additionalchart.go

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -4,9 +4,9 @@ package options
44
type AdditionalChartOptions struct {
55
// WorkingDir is the working directory for this chart within packages/<package-name>
66
WorkingDir string `yaml:"workingDir"`
7-
// UpstreamOptions is any options provided on how to get this chart from upstream. It is mutually exclusive with CRDChartOptions
7+
// UpstreamOptions is any options provided on how to get this chart from upstream.
88
UpstreamOptions *UpstreamOptions `yaml:"upstreamOptions,omitempty"`
9-
// CRDChartOptions is any options provided on how to generate a CRD chart. It is mutually exclusive with UpstreamOptions
9+
// CRDChartOptions is any options provided on how to generate a CRD chart.
1010
CRDChartOptions *CRDChartOptions `yaml:"crdOptions,omitempty"`
1111
// IgnoreDependencies drops certain dependencies from the list that is parsed from upstream
1212
IgnoreDependencies []string `yaml:"ignoreDependencies"`

templates/template/docs/packages.md

Lines changed: 0 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -16,12 +16,10 @@ additionalCharts:
1616
# These contain other charts that you would like to package alongside this chart
1717
- workingDir: # same as above
1818
upstreamOptions:
19-
# Mutually exclusive with crdOptions
2019
url: # same as above
2120
subdirectory: # optional, same as above
2221
commit: # optional, same as above
2322
crdOptions:
24-
# Mutually exclusive with upstreamOptions
2523
templateDirectory: # A directory within packages/<package>/template that will contain a template for your CRD chart
2624
crdDirectory: # Where to place your CRDs within a CRD chart (e.g. crds for default charts)
2725
addCRDValidationToMainChart: # Whether to add additional validation to your main chart to check that the CRD chart is installed.

0 commit comments

Comments
 (0)