Repository navigation
fix(receiver-naming): distinguish types with parameters - #692
Merged
Merged
Conversation
|
Hi there @mgechev! Would you mind reviewing this PR? It seems to fix a problem that I'm facing as well. Thanks! |
Collaborator
|
Hi @tie, thanks for the PR (and sorry for the late response) |
2 tasks done
subham-deepsource
pushed a commit
to DeepSourceCorp/revive
that referenced
this pull request
Aug 5, 2022
subham-deepsource
pushed a commit
to DeepSourceCorp/revive
that referenced
this pull request
Aug 5, 2022
subham-deepsource
pushed a commit
to DeepSourceCorp/revive
that referenced
this pull request
Aug 5, 2022
subham-deepsource
added a commit
to DeepSourceCorp/revive
that referenced
this pull request
Aug 5, 2022
* Separating lib from cli (revive-lint#655) * Separating lib from cli * Renamed NewRevive to New * Added GetLintFailures helper function * Moved formatter to call to format since that's when it's needed * makes fields of Revive struct non-public * minor modifs in tests: remove unnamed constats * Added lint package management to lint command * README message for using revive as a library * README formatting * Removed unused method * Slightly improved wording in README * Handling format errors * Renaming file to better reflect intent * Refactoring pattern usage * README heads * renames excludePaths into excludePatterns Co-authored-by: Bernardo Heynemann <bernardo.heynemann@coinbase.com> Co-authored-by: chavacava <salvadorcavadini+github@gmail.com> Signed-off-by: subham sarkar <subham@deepsource.io> * Update the contributors list Signed-off-by: subham sarkar <subham@deepsource.io> * Remove debugging output (revive-lint#672) Noticed during migration from our heavily modified "go-lint" to "revive" that there is an additional line printed. I am unsure that the convention for this project is on this, we do not allow adding such a call. Signed-off-by: subham sarkar <subham@deepsource.io> * Remove built-in types that existing only for the Go documentation (revive-lint#675) Since these types only exist for documenting Go's standard library there should be no reason to mark them. Closes revive-lint#673 Signed-off-by: subham sarkar <subham@deepsource.io> * Fix/677 (revive-lint#678) Signed-off-by: subham sarkar <subham@deepsource.io> * Lint cleanup (revive-lint#679) Signed-off-by: subham sarkar <subham@deepsource.io> * add rule datarace (revive-lint#683) Signed-off-by: subham sarkar <subham@deepsource.io> * Fixes issue revive-lint#619 imports-blacklist support regex (revive-lint#684) * Fixes issue revive-lint#619 imports-blacklist support regex * refactors method name and error message * restores original test cases Co-authored-by: chavacava <salvadorcavadini+github@gmail.com> Signed-off-by: subham sarkar <subham@deepsource.io> * fix(var-naming): set node to package name for underscore in package name (revive-lint#689) Setting the entire file AST as the node causes golangci-lint to print the entire file source as the context, and line and column numbers set to 1. Point to the package name node instead. Closes revive-lint#688 Signed-off-by: subham sarkar <subham@deepsource.io> * fix(deps): update module golang.org/x/tools to v0.1.11 (revive-lint#696) Co-authored-by: Renovate Bot <bot@renovateapp.com> Signed-off-by: subham sarkar <subham@deepsource.io> * fix(deps): update github.com/chavacava/garif digest to 908ad76 (revive-lint#695) Co-authored-by: Renovate Bot <bot@renovateapp.com> Signed-off-by: subham sarkar <subham@deepsource.io> * fix(receiver-naming): distinguish types with parameters (revive-lint#692) * fix(receiver-naming): distinguish types with parameters * chore: run tests using supported Go versions matrix Signed-off-by: subham sarkar <subham@deepsource.io> * Make package comment more confident (revive-lint#694) Signed-off-by: subham sarkar <subham@deepsource.io> * fix(deps): update github.com/chavacava/garif digest to d6fd61e (revive-lint#699) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Signed-off-by: subham sarkar <subham@deepsource.io> * fix issue revive-lint#691 (revive-lint#700) Signed-off-by: subham sarkar <subham@deepsource.io> * fix(deps): update github.com/chavacava/garif digest to 9351721 (revive-lint#702) Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Signed-off-by: subham sarkar <subham@deepsource.io> * Allow to customize user functions in rule `error-strings` (revive-lint#703) * Allow to customize user functions in rule `error-strings` * Rollback the Available Rules table format in README * adds memoization of the rule's configuration Co-authored-by: chavacava <salvadorcavadini+github@gmail.com> Signed-off-by: subham sarkar <subham@deepsource.io> * doc: add devlake to README usage (revive-lint#704) Co-authored-by: linyh <yanghui@meri.co> Signed-off-by: subham sarkar <subham@deepsource.io> * Check whether the tag name is duplicate or not (revive-lint#706) * Check whether the tag name is duplicate or not * - minor refactoring - continues checking tag even if name is repeated * adds test cases for duplicated tag names * adds test case with two tag types (json & yaml) * Fix allow the same tag name in different tag key * fix checks on protobuf tag names Co-authored-by: chavacava <salvadorcavadini+github@gmail.com> Signed-off-by: subham sarkar <subham@deepsource.io> * fix revive-lint#670 (revive-lint#708) Signed-off-by: subham sarkar <subham@deepsource.io> * Fix module name Signed-off-by: subham sarkar <subham@deepsource.io> Co-authored-by: Bernardo Heynemann <heynemann@gmail.com> Co-authored-by: Bernardo Heynemann <bernardo.heynemann@coinbase.com> Co-authored-by: chavacava <salvadorcavadini+github@gmail.com> Co-authored-by: mgechev <mgechev@gmail.com> Co-authored-by: Markus Zimmermann <markus.zimmermann@symflower.com> Co-authored-by: Markus Zimmermann <markus.zimmermann@nethead.at> Co-authored-by: Yudai Takada <13041216+ydah@users.noreply.github.com> Co-authored-by: Ville Skyttä <ville.skytta@iki.fi> Co-authored-by: renovate[bot] <29139614+renovate[bot]@users.noreply.github.com> Co-authored-by: Renovate Bot <bot@renovateapp.com> Co-authored-by: Ivan Trubach <mr.trubach@icloud.com> Co-authored-by: okhowang <3352585+okhowang@users.noreply.github.com> Co-authored-by: hulk <hulk.website@gmail.com> Co-authored-by: likyh <l@likyh.com> Co-authored-by: linyh <yanghui@meri.co>
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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This PR adds a new
internal/typeparamspackage that provides support for type parameters when revive is built with Go 1.18 and higher.It fixes an issue with
receiver-namingrule being unable to parse receiver type from AST method declarations the receiver has type parameters.Closes #669