chore(lint): use golangci-lint to call revive and misspell checker. #18145

appleboy · 2022-01-01T00:48:29Z

replace revive and misspell with golangci-lint

Signed-off-by: Bo-Yi Wu [email protected]

Signed-off-by: Bo-Yi Wu <[email protected]>

Gusted · 2022-01-01T01:11:32Z

Makefile

@@ -280,29 +278,6 @@ errcheck:
 	@echo "Running errcheck..."
 	@errcheck $(GO_PACKAGES)

-.PHONY: revive
-revive:


I could see why we would remove revive, I'm not sure if it actually could be a drop-in as I know from golangci-lint that they don't always have 100% configuration as if you have with cmd command.

maybe we should integrate revive in golangci-lint?

Gusted · 2022-01-01T01:15:45Z

Makefile

-		$(GO) install github.com/client9/misspell/cmd/[email protected]; \
-	fi
-	@echo "Running misspell-check..."
-	@$(GO) run build/code-batch-process.go misspell -error -i unknwon '{file-list}'


It seems to me that this shouldn't be touched, as it's using build/code-batch-process.go which seems to be special build made to batch this misspell action. Not sure why, maybe one of the other maintainers can shed some light on that.

build/code-batch-process.go is only a wrapper for CLI.

In Windows, you can not pass more than 32K bytes via CLI arguments, so thecode-batch-process.go split the file names into batches. We can just ignore the wrapper during refacotoring.

ps: the build/code-batch-process.go came from #17684 to fix #14438 , so do not worry about it 😊

Makefile

wxiaoguang · 2022-01-01T03:16:04Z

For my understanding, this PR just removes revive and misspell.

The comment said replace revive and misspell with golangci-lint. How are they replaced?

lunny · 2022-01-01T03:19:47Z

golangci-lint now use revive internally #17609 but I think misspell is still necessary.

Signed-off-by: Bo-Yi Wu <[email protected]>

appleboy · 2022-01-01T07:42:05Z

@wxiaoguang we can enable the revive lint in golangci-lint, see c96f3ce and misspell has been added in yaml file.

lunny · 2022-01-01T09:30:40Z

@wxiaoguang we can enable the revive lint in golangci-lint, see c96f3ce and misspell has been added in yaml file.

Which one is the misspell check?

appleboy · 2022-01-01T10:11:11Z

@lunny See

gitea/.golangci.yml

Line 15 in 549fd03

- misspell

codecov-commenter · 2022-01-01T10:34:41Z

Codecov Report

❗ No coverage uploaded for pull request base (main@c99b8ef). Click here to learn what that means.
The diff coverage is 45.55%.

❗ Current head 549fd03 differs from pull request most recent head c96f3ce. Consider uploading reports for the commit c96f3ce to get more accurate results

@@           Coverage Diff           @@
##             main   #18145   +/-   ##
=======================================
  Coverage        ?   45.01%           
=======================================
  Files           ?      825           
  Lines           ?    91557           
  Branches        ?        0           
=======================================
  Hits            ?    41215           
  Misses          ?    43756           
  Partials        ?     6586

Impacted Files	Coverage Δ
cmd/web_letsencrypt.go	`0.00% <0.00%> (ø)`
models/lfs.go	`23.96% <0.00%> (ø)`
models/unittest/testdb.go	`18.18% <0.00%> (ø)`
routers/web/repo/lfs.go	`0.00% <0.00%> (ø)`
services/repository/adopt.go	`26.69% <85.10%> (ø)`
modules/indexer/code/bleve.go	`66.52% <100.00%> (ø)`

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 c99b8ef...c96f3ce. Read the comment docs.

Gusted

Faster linting 🎉

zeripath · 2022-01-02T19:29:53Z

make lgtm work

* giteaoffical/main: (22 commits) Add MP4 as default allowed attachment type (go-gitea#18170) [skip ci] Updated translations via Crowdin Include folders into size cost (go-gitea#18158) Don't delete branch if other PRs with this branch are open (go-gitea#18164) Remove unused route "/tasks/trigger" (go-gitea#18160) Fix EasyMDE validation (go-gitea#18161) Fix bug (go-gitea#18168) tests: add coverage for models migration helpers (go-gitea#18162) [skip ci] Updated translations via Crowdin Require codereview to have content (go-gitea#18156) chore(lint): use golangci-lint to call revive and misspell checker. (go-gitea#18145) Update owners for 2022 (go-gitea#18155) Refactor auth package (go-gitea#17962) Unify and simplify TrN for i18n (go-gitea#18141) Use correct user when determining max repo limits for error messages (go-gitea#18153) Add singuliere to MAINTAINERS (go-gitea#18148) [skip ci] Updated licenses and gitignores Add API to get issue/pull comments and events (timeline) (go-gitea#17403) Upgrade certmagic from v0.14.1 to v0.15.2 (go-gitea#18138) Upgrade certmagic from v0.14.1 to v0.15.2 (go-gitea#18138) ...

wxiaoguang · 2022-01-08T01:19:27Z

The document should also be updated, eg: https://docs.gitea.io/en-us/hacking-on-gitea/#formatting-code-analysis-and-spell-check : make revive vet misspell-check

…o-gitea#18145) replace revive and misspell with golangci-lint Signed-off-by: Bo-Yi Wu <[email protected]>

wxiaoguang · 2022-10-16T02:07:41Z

--> Remove code-batch-process #21471

chore(lint): remove revive and misspell checker.

4a091f2

Signed-off-by: Bo-Yi Wu <[email protected]>

Gusted reviewed Jan 1, 2022

View reviewed changes

GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jan 1, 2022

wxiaoguang reviewed Jan 1, 2022

View reviewed changes

Makefile Outdated Show resolved Hide resolved

appleboy added 2 commits January 1, 2022 15:32

fix: revert two space

bc57210

Signed-off-by: Bo-Yi Wu <[email protected]>

chore(lint): enable revive lint.

c96f3ce

Signed-off-by: Bo-Yi Wu <[email protected]>

lunny approved these changes Jan 1, 2022

View reviewed changes

GiteaBot added lgtm/need 1 This PR needs approval from one additional maintainer to be merged. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Jan 1, 2022

lunny added the topic/code-linting label Jan 1, 2022

Gusted approved these changes Jan 2, 2022

View reviewed changes

GiteaBot added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 1 This PR needs approval from one additional maintainer to be merged. labels Jan 2, 2022

appleboy and others added 2 commits January 3, 2022 00:24

Merge branch 'main' into lint

0f03489

Merge branch 'main' into lint

c305050

zeripath changed the title ~~chore(lint): remove revive and misspell checker.~~ chore(lint): use golangci-lint to call revive and misspell checker. Jan 2, 2022

zeripath merged commit 948949f into go-gitea:main Jan 2, 2022

appleboy deleted the lint branch January 3, 2022 12:17

appleboy mentioned this pull request Jan 8, 2022

docs(code style): remove unnecessary section #18209

Merged

Chianina pushed a commit to Chianina/gitea that referenced this pull request Mar 28, 2022

chore(lint): use golangci-lint to call revive and misspell checker. (g…

ebd52fd

…o-gitea#18145) replace revive and misspell with golangci-lint Signed-off-by: Bo-Yi Wu <[email protected]>

go-gitea locked and limited conversation to collaborators Apr 28, 2022

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

chore(lint): use golangci-lint to call revive and misspell checker. #18145

chore(lint): use golangci-lint to call revive and misspell checker. #18145

appleboy commented Jan 1, 2022

Gusted Jan 1, 2022

appleboy Jan 1, 2022

Gusted Jan 1, 2022

wxiaoguang Jan 1, 2022

wxiaoguang Jan 1, 2022

wxiaoguang commented Jan 1, 2022

lunny commented Jan 1, 2022

appleboy commented Jan 1, 2022

lunny commented Jan 1, 2022

appleboy commented Jan 1, 2022

codecov-commenter commented Jan 1, 2022

Gusted left a comment

zeripath commented Jan 2, 2022

wxiaoguang commented Jan 8, 2022 •

edited

Loading

wxiaoguang commented Oct 16, 2022

chore(lint): use golangci-lint to call revive and misspell checker. #18145

chore(lint): use golangci-lint to call revive and misspell checker. #18145

Conversation

appleboy commented Jan 1, 2022

Gusted Jan 1, 2022

Choose a reason for hiding this comment

appleboy Jan 1, 2022

Choose a reason for hiding this comment

Gusted Jan 1, 2022

Choose a reason for hiding this comment

wxiaoguang Jan 1, 2022

Choose a reason for hiding this comment

wxiaoguang Jan 1, 2022

Choose a reason for hiding this comment

wxiaoguang commented Jan 1, 2022

lunny commented Jan 1, 2022

appleboy commented Jan 1, 2022

lunny commented Jan 1, 2022

appleboy commented Jan 1, 2022

codecov-commenter commented Jan 1, 2022

Codecov Report

Gusted left a comment

Choose a reason for hiding this comment

zeripath commented Jan 2, 2022

wxiaoguang commented Jan 8, 2022 • edited Loading

wxiaoguang commented Oct 16, 2022

wxiaoguang commented Jan 8, 2022 •

edited

Loading