Commit Graph

9 Commits

Author SHA1 Message Date
Jason Song 2b1561217f Remove DenylistImports (#23)
`DenylistImports` will be redundant once [gitea/pull/22412](https://github.com/go-gitea/gitea/pull/22412) has been merged.

Reviewed-on: https://gitea.com/gitea/gitea-vet/pulls/23
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com>
Reviewed-by: John Olheiser <john+gitea@jolheiser.com>
Co-authored-by: Jason Song <i@wolfogre.com>
Co-committed-by: Jason Song <i@wolfogre.com>
2023-01-13 10:24:36 +08:00
Jason Song 4c10dca949 Check SPDX-License-Identifier (#21)
Check `SPDX-License-Identifier` in file headers.

Related to https://github.com/go-gitea/gitea/pull/21840

Co-authored-by: Jason Song <i@wolfogre.com>
Reviewed-on: https://gitea.com/gitea/gitea-vet/pulls/21
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com>
Reviewed-by: Xinyu Zhou <i@sourcehut.net>
Co-authored-by: Jason Song <wolfogre@noreply.gitea.io>
Co-committed-by: Jason Song <wolfogre@noreply.gitea.io>
2022-12-01 23:58:05 +08:00
Gusted 791434a6a7 Add denylist import (#17)
- As per https://github.com/go-gitea/gitea/pull/18222#issuecomment-1008386067

Reviewed-on: https://gitea.com/gitea/gitea-vet/pulls/17
Reviewed-by: techknowlogick <techknowlogick@gitea.io>
Reviewed-by: Norwin <noerw@noreply.gitea.io>
Co-authored-by: Gusted <williamzijl7@hotmail.com>
Co-committed-by: Gusted <williamzijl7@hotmail.com>
2022-01-11 09:43:43 +08:00
Andrew Thornton 7c98703580 If you open a session either return it or close it (#16)
If you call `sess := x.NewSession()` you must call `sess.Close()` or return the session.

This is not quite completely perfect for example if you return the output of x.NewSession() immediately it will falsely claim that you're not closing it. However it shows the beginnings of how to do it.

Signed-off-by: Andrew Thornton <art27@cantab.net>

close #15

Reviewed-on: https://gitea.com/gitea/gitea-vet/pulls/16
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com>
Reviewed-by: 6543 <6543@obermui.de>
Co-authored-by: Andrew Thornton <art27@cantab.net>
Co-committed-by: Andrew Thornton <art27@cantab.net>
2021-04-13 06:12:04 +08:00
6543 4787511f0b Correct Import Paths (#6)
fix import path after repo move

Co-authored-by: 6543 <6543@obermui.de>
Reviewed-on: https://gitea.com/gitea/gitea-vet/pulls/6
2020-07-20 13:29:15 +00:00
John Olheiser 29eb4b7ae5 Add migrations check (#5)
Update README

Signed-off-by: jolheiser <john.olheiser@gmail.com>

Add copyright

Signed-off-by: jolheiser <john.olheiser@gmail.com>

Add migrations check

Signed-off-by: jolheiser <john.olheiser@gmail.com>

Co-authored-by: jolheiser <john.olheiser@gmail.com>
Reviewed-on: https://gitea.com/gitea/gitea-vet/pulls/5
Reviewed-by: Lunny Xiao <xiaolunwen@gmail.com>
2020-05-31 18:38:55 +00:00
John Olheiser 0185062ee4 Add CI (#1)
Add basic build when merged to master

Add basic CI compliance and linting

Signed-off-by: jolheiser <john.olheiser@gmail.com>

Co-authored-by: jolheiser <john.olheiser@gmail.com>
Reviewed-on: https://gitea.com/jolheiser/gitea-vet/pulls/1
2020-04-04 21:12:39 +00:00
jolheiser 6c7b81d401
Move checks to common package and add Makefile
Signed-off-by: jolheiser <john.olheiser@gmail.com>
2020-04-03 22:28:55 -05:00
jolheiser 92e361fc3d
Initial commit
Signed-off-by: jolheiser <john.olheiser@gmail.com>
2020-03-24 22:43:07 -05:00