refactor: replace lint to revive #5422

Merged
lunny merged 2 commits from revive into master 2018-12-03 14:28:47 +00:00
Contributor

lint time

$ time make lint
real    0m7.953s
user    0m5.928s
sys     0m2.129s

revive time

$ time make revive
revive -config config.toml -exclude=./vendor/... ./... || exit 1

real    0m5.179s
user    0m8.248s
sys     0m1.785s
## lint time ``` $ time make lint real 0m7.953s user 0m5.928s sys 0m2.129s ``` ## [revive](https://github.com/mgechev/revive) time ``` $ time make revive revive -config config.toml -exclude=./vendor/... ./... || exit 1 real 0m5.179s user 0m8.248s sys 0m1.785s ```
codecov-io commented 2018-11-28 04:16:17 +00:00 (Migrated from github.com)

Codecov Report

Merging #5422 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master    #5422   +/-   ##
=======================================
  Coverage   37.57%   37.57%           
=======================================
  Files         317      317           
  Lines       46824    46824           
=======================================
  Hits        17593    17593           
  Misses      26732    26732           
  Partials     2499     2499

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 2d707fe...8f2247d. Read the comment docs.

# [Codecov](https://codecov.io/gh/go-gitea/gitea/pull/5422?src=pr&el=h1) Report > Merging [#5422](https://codecov.io/gh/go-gitea/gitea/pull/5422?src=pr&el=desc) into [master](https://codecov.io/gh/go-gitea/gitea/commit/2d707fe5e663abb0be2127362c760446ce6cedab?src=pr&el=desc) will **not change** coverage. > The diff coverage is `n/a`. [![Impacted file tree graph](https://codecov.io/gh/go-gitea/gitea/pull/5422/graphs/tree.svg?width=650&token=t1G57YGbPy&height=150&src=pr)](https://codecov.io/gh/go-gitea/gitea/pull/5422?src=pr&el=tree) ```diff @@ Coverage Diff @@ ## master #5422 +/- ## ======================================= Coverage 37.57% 37.57% ======================================= Files 317 317 Lines 46824 46824 ======================================= Hits 17593 17593 Misses 26732 26732 Partials 2499 2499 ``` ------ [Continue to review full report at Codecov](https://codecov.io/gh/go-gitea/gitea/pull/5422?src=pr&el=continue). > **Legend** - [Click here to learn more](https://docs.codecov.io/docs/codecov-delta) > `Δ = absolute <relative> (impact)`, `ø = not affected`, `? = missing data` > Powered by [Codecov](https://codecov.io/gh/go-gitea/gitea/pull/5422?src=pr&el=footer). Last update [2d707fe...8f2247d](https://codecov.io/gh/go-gitea/gitea/pull/5422?src=pr&el=lastupdated). Read the [comment docs](https://docs.codecov.io/docs/pull-request-comments).
lunny approved these changes 2018-11-28 09:00:49 +00:00
techknowlogick reviewed 2018-11-28 15:35:35 +00:00

Is it possible to name this revive_config.toml just to prevent any confusion about what this file is for?

Is it possible to name this `revive_config.toml` just to prevent any confusion about what this file is for?
lafriks (Migrated from github.com) reviewed 2018-11-28 21:24:31 +00:00
lafriks (Migrated from github.com) commented 2018-11-28 21:24:30 +00:00

I would prefer something like .revive.toml

I would prefer something like `.revive.toml`
techknowlogick reviewed 2018-12-03 03:25:53 +00:00

@appleboy Can I make this change on your behalf?

@appleboy Can I make this change on your behalf?
sapk (Migrated from github.com) reviewed 2018-12-03 09:26:52 +00:00
sapk (Migrated from github.com) commented 2018-12-03 09:26:52 +00:00

Why not keep lint command name as it is the action (linting) not the name tool?

Why not keep lint command name as it is the action (linting) not the name tool?
appleboy reviewed 2018-12-03 14:03:58 +00:00
Author
Contributor

@techknowlogick OK, do it.

@techknowlogick OK, do it.
appleboy reviewed 2018-12-03 14:18:58 +00:00
Author
Contributor

@sapk revert.

@sapk revert.
appleboy reviewed 2018-12-03 14:19:27 +00:00
Author
Contributor

@lafriks @techknowlogick change file name to .revive.toml.

@lafriks @techknowlogick change file name to `.revive.toml`.
techknowlogick approved these changes 2018-12-03 14:20:56 +00:00
sapk (Migrated from github.com) approved these changes 2018-12-03 14:24:58 +00:00
This repo is archived. You cannot comment on pull requests.
No Milestone
No project
No Assignees
3 Participants
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: lunny/gitea#5422
No description provided.