1
0
mirror of https://github.com/mgechev/revive.git synced 2025-02-03 13:11:25 +02:00

944 Commits

Author SHA1 Message Date
haya14busa
7c17e85217 fix modifies-parameter name in README (#36) 2018-07-14 18:39:49 -07:00
mgechev
8b99937f99
Update the list of contributors 2018-07-13 15:02:51 -07:00
Markus Wiegand
095a25d375 Add formatter NDJSON (#34)
* Add formatter json-stream

* use new json struct in json-stream

* rename json-stream formatter to ndjson

* add ndjson formatter to readme
2018-07-13 15:01:27 -07:00
Markus Wiegand
1e0238d20a Add severity to json formatter (#33)
* Add severity to json formatter

* Rename struct 'JSONObject'
2018-07-12 20:36:45 -07:00
Markus Wiegand
1cab2ac566 add atom package 'linter-revive' to supported editors (#32) 2018-07-10 20:04:39 -07:00
chavacava
d95910df55 Refactoring: move pick function to rule/utils.go (#30)
* Adds rule superfluous-else (an extension of indent-error-flow)

* Fix superfluous-else rule struct namming.

* Adds superfuous-else rule to the rules table

* Adds confusing-naming rule

* adds multifile test

* [WIP] fix multiple file test

* draft solution for detecting confusing-names through multiple files

* [WIP] confusing-name multiple files

* clean-up

* draft working version

* cleaner version + more informative messages

* adds check on struct field names

* fix config.go

* clean master

* new ADS rule: newerr

* ADS-print working version

* ads-print final version

* ads-lost-err working version

* confusing-namming: fix tests

* moves pick to rule/utils for better reusability
2018-07-09 03:00:44 +10:00
mgechev
ca24398638
Update README.md 2018-07-07 15:15:12 -07:00
Minko Gechev
f966283f3e
Fix rule for documentation of exported types (#29)
* Fix rule for documentation of exported types

The rule was failing in the following case:

```go
// A is an awesome type
type A = int
```

Because revive was skipping the leading `A`, considering it as an article.

* Add "This" article and update Makefile
2018-07-08 08:14:15 +10:00
chavacava
c2e2dbac85 New rule: unused-parameter (#28)
* Adds rule superfluous-else (an extension of indent-error-flow)

* Fix superfluous-else rule struct namming.

* Adds superfuous-else rule to the rules table

* Adds confusing-naming rule

* adds multifile test

* [WIP] fix multiple file test

* draft solution for detecting confusing-names through multiple files

* [WIP] confusing-name multiple files

* clean-up

* draft working version

* cleaner version + more informative messages

* adds check on struct field names

* fix config.go

* clean master

* new ADS rule: newerr

* ADS-print working version

* ads-print final version

* ads-lost-err working version

* confusing-namming: fix tests

* unused-parameter: working version

* WIP adds scopes - still imprecise ( eg a:=a is not detected as use)

* w/scopes and more precise

* adds test on structs

* adds test w/ var shadowing

* more precise handling of for/switch statements

* fix check of +=, -=, *= and the like. Adds better support for slices and switchs
2018-07-07 18:40:02 +10:00
chavacava
b8eababb0d New rule: confusing-results (#27)
* Adds rule superfluous-else (an extension of indent-error-flow)

* Fix superfluous-else rule struct namming.

* Adds superfuous-else rule to the rules table

* Adds confusing-naming rule

* adds multifile test

* clean-up

* fix config.go

* confusing-results working version
2018-07-02 12:09:58 +10:00
chavacava
443bfc9e0b New rule: Confusing naming (#16)
* Adds rule superfluous-else (an extension of indent-error-flow)

* Fix superfluous-else rule struct namming.

* Adds superfuous-else rule to the rules table

* Adds confusing-naming rule

* adds multifile test

* [WIP] fix multiple file test

* draft solution for detecting confusing-names through multiple files

* [WIP] confusing-name multiple files

* draft working version

* cleaner version + more informative messages

* adds check on struct field names

* confusing-namming: fix tests
2018-07-02 12:05:20 +10:00
chavacava
8ffd0e86d3 Merge 'ads-lost-err'
Conflicts:
	config.go
2018-06-29 13:50:56 +02:00
chavacava
fa8bea3d3b ads-lost-err working version 2018-06-29 13:41:18 +02:00
chavacava
72bcc6911e ads-print final version 2018-06-29 13:27:12 +02:00
chavacava
a7e4e3bed3 ADS-print working version 2018-06-28 13:38:31 +02:00
chavacava
3dc86d87c8 new ADS rule: newerr 2018-06-28 11:06:43 +02:00
chavacava
27a18a3b82 Merge remote-tracking branch 'upstream/master' 2018-06-27 11:55:35 +02:00
chavacava
68deb55492 New rule deep-exit (#26)
* Adds rule superfluous-else (an extension of indent-error-flow)

* Fix superfluous-else rule struct namming.

* Adds superfuous-else rule to the rules table

* Adds confusing-naming rule

* adds multifile test

* clean-up

* fix config.go

* deep-exit: first working version

* fix pbs from  @mgechev review

* deep-exit: modifies failure message
2018-06-27 06:21:03 +10:00
chavacava
3457eee727 Merge branch 'master' of https://github.com/chavacava/revive
Conflicts:
	README.md
	config.go
2018-06-25 13:43:15 +02:00
chavacava
d1e7779818 Merge remote-tracking branch 'upstream/master' 2018-06-24 10:41:13 +02:00
chavacava
b2532b3c33 New rule: modifies-parameter (#25)
* Adds rule superfluous-else (an extension of indent-error-flow)

* Fix superfluous-else rule struct namming.

* Adds superfuous-else rule to the rules table

* Adds confusing-naming rule

* adds multifile test

* clean-up

* fix config.go

* working version
2018-06-24 17:26:21 +10:00
chavacava
7e89359269 superfuous-else handles more cases (#24)
* Adds rule superfluous-else (an extension of indent-error-flow)

* Fix superfluous-else rule struct namming.

* Adds superfuous-else rule to the rules table

* Adds confusing-naming rule

* adds multifile test

* clean-up

* fix config.go

* superflous-else now detects "jumps" caused by functions like os.Exit and log.Fatal
2018-06-24 17:24:50 +10:00
chavacava
e0a294505a fix merge conflicts 2018-06-22 20:29:51 +02:00
chavacava
f8c1094ecd New rule: get-return (#23)
* Adds rule superfluous-else (an extension of indent-error-flow)

* Fix superfluous-else rule struct namming.

* Adds superfuous-else rule to the rules table

* Adds confusing-naming rule

* adds multifile test

* clean-up

* clean master

* running version

* isGetter now checks for non lowercase characters
2018-06-22 21:21:09 +10:00
chavacava
b23f923b0e clean master 2018-06-21 16:43:34 +02:00
Salvador Cavadini
1538d386e6 fix config.go 2018-06-20 13:14:44 +02:00
Salvador Cavadini
eabfc8aae9 Merge branch 'master' of https://github.com/chavacava/revive 2018-06-20 13:12:50 +02:00
Salvador Cavadini
b8d77e5a08 Merge branch 'master' of https://github.com/mgechev/revive 2018-06-20 13:10:53 +02:00
mgechev
a1d3163745
Update the list of contributors 2018-06-14 17:31:32 -07:00
Veniamin Krol
83bd86be4f Fix typo in README.md (#21) 2018-06-13 12:35:38 -07:00
chavacava
4e19174270 Fix issue19 (#20)
* Adds rule superfluous-else (an extension of indent-error-flow)

* Fix superfluous-else rule struct namming.

* Adds superfuous-else rule to the rules table

* fix issue 19
2018-06-13 07:01:31 -07:00
Salvador Cavadini
ca1b61fcaa Merge remote-tracking branch 'upstream/master' 2018-06-13 08:39:06 +02:00
Salvador Cavadini
6ed9d2ca06 clean-up 2018-06-13 08:23:16 +02:00
Salvador Cavadini
b34b9c4861 Merge branch 'master' of https://github.com/mgechev/revive 2018-06-13 07:55:37 +02:00
chavacava
4f2e0613fc adds multifile test 2018-06-12 06:54:21 +02:00
chavacava
c9bde6c503 empty-block: ignore checking blocks of funcs and func literals (#17)
* Adds rule superfluous-else (an extension of indent-error-flow)

* Fix superfluous-else rule struct namming.

* Adds superfuous-else rule to the rules table

* Modifies empty-block rule to ignore bodies of
func literals and funcs

* add test cases on functions with a receiver for completeness
2018-06-11 12:08:58 -07:00
Tamir Duberstein
32df8ca880 Avoid false positives when returning interface values (#6)
* Use subtests

* Make unexported-return type check

* Avoid false positives when returning interface values

Fixes #5.
2018-06-11 11:22:33 -07:00
Tamir Duberstein
1b2ffe282e Improve tests (#18)
* Use subtests

* Make unexported-return type check
2018-06-11 11:22:06 -07:00
chavacava
97315129a9 Adds confusing-naming rule 2018-06-10 13:25:46 +02:00
mgechev
fb3c2d09af
Update list of contributors 2018-06-09 13:18:47 -07:00
chavacava
b7bb0314ca resolves conflict in README.md 2018-06-09 18:25:45 +02:00
mgechev
8198433610
Update readme with vim support 2018-06-08 16:04:26 -07:00
mgechev
1b84ff83dd
Add missing rule in config 2018-06-08 14:22:21 -07:00
chavacava
1fa5046357 Adds new rule empty-block (#14)
* Adds rule superfluous-else (an extension of indent-error-flow)

* initial (non functional) version

* empty-block working version

* adds tests for empty-block rule

* Adds empty-block to the rules table

* code clean-up
2018-06-08 12:41:49 -07:00
mgechev
dc6767b811
Add toc 2018-06-08 10:28:17 -07:00
mgechev
05db4673db
Add list of contributors 2018-06-08 10:24:28 -07:00
chavacava
cbe45ffc79 Adds rule superfluous-else (an extension of indent-error-flow) (#13)
* Adds rule superfluous-else (an extension of indent-error-flow)

* Fix superfluous-else rule struct namming.

* Adds superfuous-else rule to the rules table
2018-06-08 07:06:29 -07:00
Salvador Cavadini
409f3e1179 Adds superfuous-else rule to the rules table 2018-06-08 13:12:29 +02:00
Salvador Cavadini
336f3e0750 Fix superfluous-else rule struct namming. 2018-06-07 13:54:43 +02:00
Salvador Cavadini
ee3547ea90 Merge remote-tracking branch 'upstream/master' 2018-06-07 13:30:17 +02:00