-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
feat: new custom linters system #4437
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
Merged
Merged
Changes from all commits
Commits
Show all changes
26 commits
Select commit
Hold shift + click to select a range
15c4050
feat: POC
ldez a9287ee
Revert "feat: POC"
ldez cd7fa6d
docs: create pages
ldez e58d90f
chore: ignore plugin artifacts
ldez a0a31ba
feat: update plugin configuration
ldez bf297c3
chore: rename builder_plugin.go to builder_plugin_go.go
ldez bc04824
chore: rename PluginBuilder to PluginGoBuilder
ldez 7a5068b
feat: add new plugin builder (db)
ldez f318ed4
chore: add file to declare plugins
ldez 4543af4
feat: add new command
ldez 7848198
docs: add mygcl configuration file
ldez 942725a
docs: update .golangci.reference.yml
ldez 9f9eb2f
feat: use the new plugin builder inside the commands
ldez 89333a8
docs: add requirements
ldez a0bd603
feat: change Builder interface to return error
ldez 59c30d1
feat: filter to plugin type
ldez faff4e0
feat: support yml, yaml, json as file extension
ldez 78280f7
chore: remove useless call to plugin builder
ldez d1f8a3f
tests: add tests on LintersSettings
ldez 27334e8
tests: add tests on Configuration
ldez b7e762b
tests: add tests on generateImports
ldez 664b6ef
chore: remove useless call to plugin builder
ldez 0d7e33b
docs: add linter example
ldez 00b90bc
review: verbose command
ldez 6f971ed
review: sanitize and comment
ldez e13d2fa
review: mygcl -> custom-gcl
ldez File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,37 @@ | ||
# The golangci-lint version used to build the custom binary. | ||
# Require. | ||
version: v1.56.2 | ||
|
||
# the name of the custom binary. | ||
# Optional. | ||
# Default: custom-gcl | ||
name: custom-golangci-lint | ||
|
||
# The directory path used to store the custom binary. | ||
# Optional. | ||
# Default: . | ||
destination: ./my/path/ | ||
|
||
# The list of the plugins to integrate inside the custom binary. | ||
plugins: | ||
# a plugin from a Go proxy | ||
- module: 'github.com/example/plugin3' | ||
version: v1.2.3 | ||
|
||
# a plugin from a Go proxy (with a specific import path) | ||
- module: 'github.com/example/plugin4' | ||
import: 'github.com/example/plugin4/foo' | ||
version: v1.0.0 | ||
|
||
# a plugin from local source (with absolute path) | ||
- module: 'github.com/example/plugin2' | ||
path: /my/local/path/plugin2 | ||
|
||
# a plugin from local source (with relative path) | ||
- module: 'github.com/example/plugin1' | ||
path: ./my/local/path/plugin1 | ||
|
||
# a plugin from local source (with absolute path and a specific import path) | ||
- module: 'github.com/example/plugin2' | ||
import: 'github.com/example/plugin4/foo' | ||
path: /my/local/path/plugin2 |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -18,3 +18,7 @@ | |
/vendor/ | ||
coverage.out | ||
coverage.xml | ||
/custom-golangci-lint | ||
/custom-gcl | ||
.custom-gcl.yml | ||
.custom-gcl.yaml |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,3 @@ | ||
package main | ||
|
||
// This file is used to declare module plugins. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,76 @@ | ||
--- | ||
title: Go Plugin System | ||
--- | ||
|
||
Private linters can be added through [Go's plugin system](https://pkg.go.dev/plugin). | ||
|
||
For a private linter (which acts as a plugin) to work properly, | ||
the plugin as well as the golangci-lint binary **needs to be built for the same environment**. | ||
|
||
`CGO_ENABLED` is another requirement. | ||
|
||
This means that `golangci-lint` needs to be built for whatever machine you intend to run it on | ||
(cloning the golangci-lint repository and running a `CGO_ENABLED=1 make build` should do the trick for your machine). | ||
|
||
## Create a Plugin | ||
|
||
Your linter must provide one or more `golang.org/x/tools/go/analysis.Analyzer` structs. | ||
|
||
Your project should also use `go.mod`. | ||
|
||
All versions of libraries that overlap `golangci-lint` (including replaced libraries) MUST be set to the same version as `golangci-lint`. | ||
You can see the versions by running `go version -m golangci-lint`. | ||
|
||
You'll also need to create a Go file like `plugin/example.go`. | ||
|
||
This file MUST be in the package `main`, and MUST define an exposed function called `New` with the following signature: | ||
```go | ||
func New(conf any) ([]*analysis.Analyzer, error) { | ||
// ... | ||
} | ||
``` | ||
|
||
See [plugin/example.go](https://github.com/golangci/example-plugin-linter/blob/master/plugin/example.go) for more info. | ||
|
||
To build the plugin, from the root project directory, run: | ||
```bash | ||
go build -buildmode=plugin plugin/example.go | ||
``` | ||
|
||
This will create a plugin `*.so` file that can be copied into your project or another well known location for usage in `golangci-lint`. | ||
|
||
## Configure a Plugin | ||
|
||
If you already have a linter plugin available, you can follow these steps to define its usage in a projects `.golangci.yml` file. | ||
|
||
An example linter can be found at [here](https://github.com/golangci/example-plugin-linter). | ||
|
||
If you're looking for instructions on how to configure your own custom linter, they can be found further down. | ||
|
||
1. If the project you want to lint does not have one already, copy the [.golangci.yml](https://github.com/golangci/golangci-lint/blob/master/.golangci.yml) to the root directory. | ||
2. Adjust the yaml to appropriate `linters-settings.custom` entries as so: | ||
```yaml title=.golangci.yml | ||
linters-settings: | ||
custom: | ||
example: | ||
path: /example.so | ||
description: The description of the linter | ||
original-url: github.com/golangci/example-linter | ||
settings: # Settings are optional. | ||
one: Foo | ||
two: | ||
- name: Bar | ||
three: | ||
name: Bar | ||
``` | ||
|
||
That is all the configuration that is required to run a custom linter in your project. | ||
|
||
Custom linters are enabled by default, but abide by the same rules as other linters. | ||
|
||
If the disable all option is specified either on command line or in `.golang.yml` files `linters.disable-all: true`, custom linters will be disabled; | ||
they can be re-enabled by adding them to the `linters.enable` list, | ||
or providing the enabled option on the command line, `golangci-lint run -Eexample`. | ||
|
||
The configuration inside the `settings` field of linter have some limitations (there are NOT related to the plugin system itself): | ||
we use Viper to handle the configuration but Viper put all the keys in lowercase, and `.` cannot be used inside a key. |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,71 @@ | ||
--- | ||
title: Module Plugin System | ||
--- | ||
|
||
An example linter can be found at [here](https://github.com/golangci/example-plugin-module-linter/settings). | ||
ldez marked this conversation as resolved.
Show resolved
Hide resolved
|
||
|
||
## The Automatic Way | ||
|
||
- define your building configuration into `.custom-gcl.yml` | ||
- run the command `golangci-lint custom` ( or `golangci-lint custom -v` to have logs) | ||
- define the plugin inside the `linters-settings.custom` section with the type `module`. | ||
- run your custom version of golangci-lint | ||
|
||
Requirements: | ||
- Go | ||
- git | ||
|
||
### Configuration Example | ||
|
||
```yaml title=.custom-gcl.yml | ||
version: v1.57.0 | ||
plugins: | ||
# a plugin from a Go proxy | ||
- module: 'github.com/golangci/plugin1' | ||
import: 'github.com/golangci/plugin1/foo' | ||
version: v1.0.0 | ||
|
||
# a plugin from local source | ||
- module: 'github.com/golangci/plugin2' | ||
path: /my/local/path/plugin2 | ||
``` | ||
|
||
```yaml title=.golangci.yml | ||
linters-settings: | ||
custom: | ||
foo: | ||
type: "module" | ||
description: This is an example usage of a plugin linter. | ||
settings: | ||
message: hello | ||
|
||
linters: | ||
disable-all: true | ||
enable: | ||
- foo | ||
``` | ||
|
||
## The Manual Way | ||
|
||
- add a blank-import of your module inside `cmd/golangci-lint/plugins.go` | ||
- run `go mod tidy`. (the module containing the plugin will be imported) | ||
- run `make build` | ||
- define the plugin inside the configuration `linters-settings.custom` section with the type `module`. | ||
ldez marked this conversation as resolved.
Show resolved
Hide resolved
|
||
- run your custom version of golangci-lint | ||
|
||
### Configuration Example | ||
|
||
```yaml title=.golangci.yml | ||
linters-settings: | ||
custom: | ||
foo: | ||
type: "module" | ||
description: This is an example usage of a plugin linter. | ||
settings: | ||
message: hello | ||
|
||
linters: | ||
disable-all: true | ||
enable: | ||
- foo | ||
``` |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Uh oh!
There was an error while loading. Please reload this page.