Skip to content

chore(ci): move go lint and deps to taskfile#23

Open
odiora wants to merge 3 commits intomainfrom
chore/ci/move-go-lint-and-deps-to-taskfile
Open

chore(ci): move go lint and deps to taskfile#23
odiora wants to merge 3 commits intomainfrom
chore/ci/move-go-lint-and-deps-to-taskfile

Conversation

@odiora
Copy link
Copy Markdown
Collaborator

@odiora odiora commented Apr 23, 2026

No description provided.

Signed-off-by: Polina Sizintseva <polina.sizintseva@flant.com>
@odiora odiora force-pushed the chore/ci/move-go-lint-and-deps-to-taskfile branch from b52b786 to 1df9b49 Compare April 23, 2026 08:54
odiora added 2 commits April 23, 2026 12:00
Signed-off-by: Polina Sizintseva <polina.sizintseva@flant.com>
Signed-off-by: Polina Sizintseva <polina.sizintseva@flant.com>
@odiora odiora requested a review from Fral738 April 24, 2026 08:39
Comment thread Taskfile.yaml
deckhouse_lib_helm_ver: 1.71.2
target: ""
VALIDATION_FILES: "tools/validation/{main,messages,diff,doc_changes}.go"
prettier_image: 'ghcr.io/deckhouse/virtualization/prettier:3.2.5'
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

replace image with public prettier

Comment thread Taskfile.yaml
cmds:
- test -x "{{.golangci_lint_bin}}" || (echo "golangci-lint not found at {{.golangci_lint_bin}}. Run 'task deps:install:golangci-lint' or 'task deps:install:all'."; exit 1)

format:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

can we use our common-ci here for format?

Comment thread Taskfile.yaml
# TODO: update image reference
- task: format:yaml

_prettier:yaml:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same

Comment thread Taskfile.yaml
- task: lint:doc-ru
- task: lint:prettier:yaml

lint:go:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we use lint from our common-ci?

Comment thread Taskfile.yaml
@@ -80,11 +265,9 @@ tasks:
lint:prettier:yaml:
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Same

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants