-
-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs: add GitLab documentation #5497
Conversation
Hey, thank you for opening your first Pull Request ! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hello,
I think the content of "CI Integration" should be replaced by a link to https://golangci-lint.run/welcome/install/#ci-installation
And your modifications should be moved to this section.
Just added one additional note about the domain of CI components, just so that self-hosted users do not have to search for that in the GitLab documentation first. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Since there's some kind of official documentation for this, I decided to link it.
edit: we could even link to the official "CI/CD component" if wanted.