msb-public/helm - helm - 马士兵教育代码仓库

Commit Graph

Author	SHA1	Message	Date
Terry Howe	d262c7ca79	Merge pull request #32296 from mmorel-35/gocritic-singleCaseSwitch chore: fix emptyStringTest, nestingReduce and singleCaseSwitch checks issues from gocritic	2 months ago
Matthieu MOREL	96b315f188	chore: fix emptyStringTest, nestingReduce and singleCaseSwitch checks issues from gocritic Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	2 months ago
Matthieu MOREL	4ade4a5495	test: replace assert.Contains with assert.ErrorContains for better error handling Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	2 months ago
Matthieu MOREL	f34ad6c337	chore: fix several checks issues from testifylint Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	2 months ago
Matthieu MOREL	07259ecc86	chore: fix several checks issues from gocritic Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	2 months ago
Matthieu MOREL	34cfdfdbe5	chore: enable contextcheck, fatcontext and noctx linters Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	3 months ago
George Jenkins	253da5106a	Merge pull request #32171 from mmorel-35/empty-len Fix empty and len testifylint violations across test files	3 months ago
George Jenkins	81d6f04c82	Merge pull request #30867 from mmorel-35/gocritic chore: fix deprecatedComment issues from gocritic	3 months ago
Terry Howe	59b57c5c31	Merge pull request #31885 from mmorel-35/whitespace chore: fix whitespace linter	3 months ago
Matthieu MOREL	488c4a805d	Fix empty and len testifylint violations across test files Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	4 months ago
Evans Mungai	d374e823b2	Merge pull request #32086 from mmorel-35/error-is-as testifylint: enable error-is-as and error-nil rules	4 months ago
Aleksei Sviridkin	979e68fbbd	internal/plugin: remove zero-width spaces from plugin name comment The comment describing allowed plugin name characters contained three U+200B (zero-width space) characters around the '_' and '-' literals. The rendered comment is identical without them; the ZWSP were likely copy-pasted from a rich-text source. Downstream impact: every project that vendors helm.sh/helm/v4 and runs Renovate gets a repo-wide warning on its Dependency Dashboard about hidden Unicode characters. Renovate scans the whole tree (including vendor/) for ZWSP/bidi-override codepoints and cannot be told to skip a path for this specific check. Removing the characters here clears the warning everywhere downstream. Signed-off-by: Aleksei Sviridkin <f@lex.la>	4 months ago
Matthieu MOREL	f8ba28bb17	testifylint: enable error-is-as and error-nil rules Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	5 months ago
George Jenkins	087736b66e	fix: unnecessary-format lint issues from merge Signed-off-by: George Jenkins <gvjenkins@gmail.com>	5 months ago
George Jenkins	586eb57338	fix: Plugin missing provenance bypass Signed-off-by: George Jenkins <gvjenkins@gmail.com>	5 months ago
George Jenkins	f8afb35f4e	Merge commit from fork fix: Plugin version path traversal	5 months ago
Matthieu MOREL	7edfff33eb	chore: fix unnecessary-format issues from revive Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
Matthieu MOREL	83a0138172	chore: fix deprecatedComment issues from gocritic Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
Matthieu MOREL	1e0f702f00	chore: fix whitespace linter Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	6 months ago
George Jenkins	47a084091e	ignore error plugin loads (cli, getter) Signed-off-by: George Jenkins <gvjenkins@gmail.com>	6 months ago
George Jenkins	36dcc27ca3	fix: Plugin version path traversal Signed-off-by: George Jenkins <gvjenkins@gmail.com>	6 months ago
tsinglua	259f181808	chore: fix some minor issues in the comments Signed-off-by: tsinglua <tsinglua@outlook.com>	6 months ago
George Jenkins	c4392033cc	Merge pull request #31853 from atombrella/feature/fmt_unneeded Replace non-format use of t.Fatalf with t.Fatal	6 months ago
Matthieu MOREL	d4f6193a7e	chore(internal): enable perfsprint linter (#31871 ) #### Description enable perfsprint linter in internal/plugin Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	7 months ago
Matthieu MOREL	9409226e15	chore(internal): enable perfsprint linter #### Description enable perfsprint linter in internal/plugin/installer Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	7 months ago
Mads Jensen	36cb3a2fe9	Replace unneeded use of t.Fatalf with t.Fatal Signed-off-by: Mads Jensen <atombrella@users.noreply.github.com>	7 months ago
Matthieu MOREL	5b6c6bbfc7	fix: enable nolinlint linter Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	7 months ago
George Jenkins	bc43812115	Merge pull request #31807 from mmorel-35/errorlint-internal-3-48c7a5a fix(internal): errorlint linter	7 months ago
Matthieu MOREL	025418291a	fix(internal): errorlint linter errorlint linter in internal/third_party/dep/fs Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	7 months ago
Matthieu MOREL	abecafa0f5	fix(internal): errorlint linter #### Description errorlint linter in internal Signed-off-by: Matthieu MOREL <matthieu.morel35@gmail.com>	7 months ago
Terry Howe	429ce93848	Merge pull request #31613 from aslafy-z/feat/getter-env fix(getter): pass settings environment variables	8 months ago
Gergely Brautigam	138f730aac	fix: typo in the function names Signed-off-by: Gergely Brautigam <182850+Skarlso@users.noreply.github.com>	8 months ago
Mads Jensen	a490bb3c20	Use errors.Is to check for io.EOF and gzip.ErrHeader In GoLang, using the == operator to check for a certain error will not unwrap the error chain, and therefore may hide the problem. Signed-off-by: Mads Jensen <atombrella@users.noreply.github.com>	9 months ago
Terry Howe	b52b6615f0	Merge pull request #31207 from zyfy29/delete-unused-var chore: delete unused var in installer.go	9 months ago
Terry Howe	d28853e206	Merge pull request #31624 from atombrella/feature/sloglint Enable the `sloglint` linter	9 months ago
Terry Howe	642462e9fe	Merge pull request #11991 from twz123/no-bash-in-testscripts Make test scripts run without /bin/bash	9 months ago
Mads Jensen	a18e59e465	Enable the sloglint linter Signed-off-by: Mads Jensen <atombrella@users.noreply.github.com>	9 months ago
Zadkiel AHARONIAN	119341dca7	fix(getter): pass settings environment variables Signed-off-by: Zadkiel AHARONIAN <hello@zadkiel.fr>	9 months ago
Scott Rigby	4d54bea5ff	Merge pull request #31491 from benoittgt/31490-plugin-name-helper feat: improve plugin name validation error messages and field name detection (v1)	10 months ago
Benoit Tigeot	71591ee63e	style: linting Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	10 months ago
Benoit Tigeot	b296cbef6c	test: split tests between valid and invalid Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	10 months ago
Benoit Tigeot	9b242dd9ed	test: convert tests to table drive tests Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	10 months ago
Benoit Tigeot	c81a09b89a	test: refactor TestMetadataLegacyValidate to be more generic Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	10 months ago
yxxhero	8c87024860	update tests Signed-off-by: yxxhero <aiopsclub@163.com>	10 months ago
yxxhero	2d49f0cb4a	Refactor environment variable expansion in PrepareCommands and update tests Signed-off-by: yxxhero <aiopsclub@163.com>	10 months ago
Benoit Tigeot	acf331a005	fix: improve plugin name validation err messages early via unmarshalling - Add strict YAML unmarshalling for v1 plugins (d.KnownFields) - Add comprehensive test coverage for validPluginName regex - Maintain backwards compatibility for legacy plugins Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	10 months ago
Benoit Tigeot	9e1e3d21c5	fix: Make invalid name error message more similar and move tests Follow Scott comment\ https://github.com/helm/helm/pull/31491/files#r2524933784 Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	10 months ago
Benoit Tigeot	cf077ceb27	fix: focus only on plugin name but give more info about what we get And improve test Follow Scott comment: > here I would have a list of valid and invalid names based on pattern, with a check and intended outcome for each one so that we comprehensively cover the rules. https://github.com/helm/helm/pull/31491/files#r2524820312 Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	10 months ago
Benoit Tigeot	f4b139a82c	Make validation error similar and explicit for both metadatas Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	10 months ago
Benoit Tigeot	c04e18e452	fix: improve plugin name validation error messages Signed-off-by: Benoit Tigeot <benoit.tigeot@lifen.fr>	10 months ago

1 2

87 Commits (copilot/fix-analyze-go-job)