-
Notifications
You must be signed in to change notification settings - Fork 16
Closed
Labels
Difficulty/2:MediumMedium ticketMedium ticketGood First IssueSuitable for first-time contributorsSuitable for first-time contributorsHacktoberfestHelp WantedCommunity help wantedCommunity help wantedTech/GoType/TaskA general taskA general task
Description
Mattermost is currently in the process of using golangci-lint as the default linter for all go projects. mattermost-community/mattermost-plugin-autolink#108 serves as an example of how the migration should look like:
- Copy
.golangci.ymlfrom https://github.com/mattermost/mattermost-plugin-autolink/ into this repository. - Copy
Makefilefrom https://github.com/mattermost/mattermost-plugin-autolink/ into this repository. - Copy
build/from https://github.com/mattermost/mattermost-plugin-autolink/ into this repository. - Run
go mod tidyandmake distto ensure the build system works fine. - Update
linters-settings.goimports.local-prefixesto the go module path i.e. togithub.com/mattermost/mattermost-plugin-aws-SNS. - Run
make golangci-lintand fix all issues that arise.
Fixing an linter issue is sometimes trivial but can also be quite complicated. There might be cases, where ignoring an issue using issues.exclude-rules is needed. If you don't know how to resolve an error, please reach out to @hanzei on the Mattermost Community Server or ask the question here.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
Difficulty/2:MediumMedium ticketMedium ticketGood First IssueSuitable for first-time contributorsSuitable for first-time contributorsHacktoberfestHelp WantedCommunity help wantedCommunity help wantedTech/GoType/TaskA general taskA general task