1
0
mirror of https://github.com/mgechev/revive.git synced 2024-11-24 08:32:22 +02:00
Commit Graph

255 Commits

Author SHA1 Message Date
AragurDEV
50f3790f32 Add gitea to "Who uses Revive" (#94)
When merged: https://github.com/go-gitea/gitea/pull/5422 gitea will use revive.
2018-12-04 08:45:11 -08:00
Rıdvan Şümşet
1338c47ef1 Update README.md (#95) 2018-12-04 08:42:55 -08:00
Minko Gechev
b4b876c347 docs: update readme 2018-11-25 17:56:04 -08:00
SalvadorC
47d2b15cf9
Merge pull request #92 from jamesmaidment/fix-line-length-doc
Fix spelling of length (Closes #91, related to #93)
2018-11-06 07:09:04 +01:00
SalvadorC
623ef04eb0
Merge pull request #93 from xuri/miscellaneous
Doc updated: fix typo (see #91 and #92)
2018-11-06 07:07:17 +01:00
xuri
9fccf1407f
Doc updated: fix typo 2018-11-06 10:28:24 +08:00
James Maidment
6b24cd60a8 Fix spelling of length 2018-11-05 17:41:25 -05:00
mgechev
26ec4a5367
ci: update travis scripts 2018-10-31 12:55:27 -07:00
SalvadorC
55fe366696 call-to-gc (new rule) (#90) 2018-10-31 07:32:23 -07:00
mgechev
ad7df7dc37
ci: remove apt install 2018-10-30 16:35:41 -07:00
mgechev
46944678f0
ci: update travis config 2018-10-30 16:32:27 -07:00
mgechev
7391637be6
ci: update deployment script 2018-10-30 16:27:32 -07:00
mgechev
1eba0c6f7b
ci: update deployment script 2018-10-30 16:16:17 -07:00
mgechev
2020b30eef
feat: add links to rules in formatters
This PR introduces:

- A new formatter called "plain" which produces the same output as "default" but also appends link to the rule
- The "friendly" and "stylish" formatters now have URLs to the rules instead of the rule names
- Update of the README, adding the "unix" and "plain" formatters
2018-10-30 16:07:32 -07:00
mgechev
376425d517
refactor: remove URL field from the failure struct 2018-10-30 15:37:27 -07:00
mgechev
7845c8af5a
infra: update commit message 2018-10-30 15:27:44 -07:00
mgechev
a3410192a6
infra: update git settings 2018-10-30 15:24:34 -07:00
mgechev
15e9397722
infra: update dir name 2018-10-30 15:19:49 -07:00
mgechev
33a4e4354c
infra: introduce auto docs deployment 2018-10-30 15:17:49 -07:00
mgechev
240d9ef0b2
Optimize the logo 2018-10-30 12:59:24 -07:00
SalvadorC
1dbda0e473
Merge pull request #88 from mgechev/minko/update-docs
docs: update the documentation
2018-10-26 07:48:50 +02:00
mgechev
8e24648520
docs: update the documentation
I did some formatting:

- There were new lines in the middle of few sentences
- Added newlines before code blocks
- Removed trailing spaces

I also dropped statements in the form "This rule checks/does/warns...", when this was clear from the rule's explanation.
2018-10-25 20:29:31 -07:00
SalvadorC
3ef16e4c45 Doc rules description (#87)
* partial document

* complete list of rules descriptions

* Links rules in README to corresponding descriptions

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Co-Authored-By: chavacava <salvadorcavadini+github@gmail.com>

* Update RULES_DESCRIPTIONS.md

Includes @mgechev remarks
2018-10-25 20:15:13 -07:00
xuri
3c177e2824 New rule: line-length-limit (#86)
* New rule: line-length-limit

* simplify the panic message of rule line-length-limit

* Refactors `line-length-limit` rule, use private `check` method instead of `Visit`
2018-10-19 08:18:33 -04:00
mgechev
4b3d324865
docs: update the list of contributors 2018-10-17 10:07:33 -07:00
SalvadorC
6633ce6f24
Merge pull request #85 from gsamokovarov/empty-lines-comments-check
Fix the same-line comments detection in the empty-lines rule
2018-10-13 20:15:31 +02:00
SalvadorC
3cae2fa6f9
Update empty-lines.go
removes warning on unused parameter
2018-10-13 20:13:29 +02:00
Genadi Samokovarov
b7d9bce0b6 Fix the same-line comments detection in the empty-lines rule
Fixes #84.
2018-10-13 09:47:29 -07:00
Genadi Samokovarov
04516d2f82 Handle multi-line statements edge-cases in the multi-line rule (#83)
There were many cases where the multi-line statements at the end of a
block were marked as containing empty lines, while they didn't. Or vice
versa.
2018-10-12 22:38:08 -07:00
mgechev
69f012a7d9
Merge master 2018-10-11 14:55:17 -07:00
mgechev
892cedd764
docs: add description for empty-lines 2018-10-11 14:54:27 -07:00
Genadi Samokovarov
d2e0e5a0ed Introduce the empty-lines rule that verifies there are no {head,trail}ing lines (#81) 2018-10-11 14:52:46 -07:00
SalvadorC
da63d0a965 Refactors atomic rule code to move gofmt function to utils.go (issue #79) (#80)
* Refactoring on atomic rule:
-Main modification: move func gofmt to utils.go

* Refactoring on constant-logical-expr rule
Simplifies equality check of subexpressions by using gofmt function
2018-10-03 11:56:57 -07:00
SalvadorC
e9013628d7
Merge pull request #78 from xuri/miscellaneous
Fix rule name in README, and format code.
2018-10-03 14:55:27 +02:00
xuri
7d2c43c827
Rename rule name waitgroup-by-copy to waitgroup-by-value 2018-10-03 16:01:41 +08:00
xuri
c781c8b090
Fix rule name in README, and format code. 2018-10-03 14:12:19 +08:00
Minko Gechev
cd6459542b
docs: add pull request template 2018-10-01 12:34:37 -07:00
xuri
9545122d39 New rule: atomic (#77) 2018-10-01 12:24:17 -07:00
SalvadorC
6ebe5bbb1e waitgroup-by-value (new rule) (#76)
* waitgroup-by-value (new rule)

* code simplification
2018-09-30 12:29:11 -07:00
SalvadorC
672b7cdf37
Merge pull request #75 from xuri/range-loop-var
New rule: range-loop-var
2018-09-29 07:07:33 +02:00
SalvadorC
f3834d0604
Removes linter warning
Renamed unused parameter `arguments` to `_`
2018-09-29 07:04:10 +02:00
xuri
924e3db3ba
Rename rule range-loop-var to range-val-in-closure. 2018-09-29 11:32:32 +08:00
xuri
b34a6c053e
New rule: range-loop-var 2018-09-27 23:21:10 +08:00
SalvadorC
388baa3eeb Fix linter warnings (recommended config) in the code (#71) 2018-09-22 15:27:22 -07:00
SalvadorC
bd4139713d fix issue [#59] (#70) 2018-09-21 10:51:01 -07:00
mgechev
0c79a9c405
docs: update readme 2018-09-20 10:47:03 -07:00
Pavel Sapezhko
20b1f54a85 Add missed max-public-structs rule to all list (#69) 2018-09-18 08:19:25 -07:00
Sylvain Kerkour
02575a7674 Add the imports-blacklist rule (#68)
* Add the imports-blacklist rule with tests

* Add the imports-blacklist rule to README.md

* Update the imports-blacklist rule to use map[string]bool so that we can have the algorithm in Visit below with O(n) instead of O(n*m)

* Fix the imports-blacklist rule to be case sensitive
2018-09-17 13:06:42 -07:00
SalvadorC
7240bd56d0
Merge pull request #62 from xuri/return-value-limit
New rule: return-result-limit
2018-09-17 21:56:59 +02:00
mgechev
21b7654079
docs: update the list of contributors 2018-09-17 12:40:59 -07:00