7 Commits (copilot/important-issues-review)

Author	SHA1	Message	Date
Abhay Chaurasiya	82be04d7f2	remove legacy import comments from remaining packages (#31933) ... Pre-Go-modules import path comments (e.g. `// import "helm.sh/helm/v4/..."`) are obsolete since Go 1.11 where go.mod is the authoritative module path. These stale comments cause issues with downstream tooling such as Kythe. Removes the legacy import comments from 64 files across 20 packages: - cmd/helm - internal/chart/v3/lint and sub-packages - internal/plugin and sub-packages - internal/release/v2/util - pkg/chart/v2/lint and sub-packages - pkg/cmd, pkg/engine, pkg/ignore, pkg/provenance - pkg/registry - pkg/release/v1/util - pkg/repo/v1 - pkg/storage and pkg/storage/driver Follows up on #31931 and #31932 which addressed pkg/kube. Fixes #31846 Signed-off-by: abhay1999 <abhay.chaurasiya@example.com> Signed-off-by: abhay1999 <abhaychaurasiya19@gmail.com>	4 weeks ago
Matthieu MOREL	a02a5713de	chore: fix elseif and ifElseChain issues from gocritic (#32289) ... * chore: fix elseif and ifElseChain issues from gocritic Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com> * test: enhance assertions in OCIInstaller and list tests using testify Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com> --------- Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	1 month ago
Matthieu MOREL	96b315f188	chore: fix emptyStringTest, nestingReduce and singleCaseSwitch checks issues from gocritic ... Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	1 month ago
Matthieu MOREL	abecafa0f5	fix(internal): errorlint linter ... #### Description errorlint linter in internal Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
Matthieu MOREL	e2d184c79e	chore(internal): fix modernize linter ... #### Description fix modernize linter in internal/chart/v3/lint/rules Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
Kamil Swiechowski	cd76ae1c93	feat:strict compliance with semverv2 for chart/v3/linter ... Signed-off-by: Kamil Swiechowski <kamil.swiechowski@iprally.com>	12 months ago
Matt Farina	9dcc49cbd5	Move lint pkg to be part of each chart version ... Linting is specific to the chart versions. A v2 and v3 chart will lint differently. To accomplish this, packages like engine need to be able to handle different chart versions. This was accomplished by some changes: 1. The introduction of a Charter interface for charts 2. The ChartAccessor which is able to accept a chart and then provide access to its data via an interface. There is an interface, factory, and implementation for each version of chart. 3. Common packages were moved to a common and util packages. Due to some package loops, there are 2 packages which may get some consolidation in the future. The new interfaces provide the foundation to move the actions and cmd packages to be able to handle multiple apiVersions of charts. Signed-off-by: Matt Farina <matt.farina@suse.com>	12 months ago