Add gitea-vet #121

Merged
zeripath merged 10 commits from 6543/tea:add-gitea-vet into master 2 years ago
6543 commented 2 years ago
Collaborator

close #119

close #119
6543 added the
status/wip
kind/build
labels 2 years ago
Collaborator

@6543 Having gitea-vet in the gitignore means anyything that matches gitea-vet is ignored, including the vendor directory.

Just needed to change it to /gitea-vet to exclude only the binary.

@6543 Having `gitea-vet` in the gitignore means _anyything_ that matches `gitea-vet` is ignored, including the vendor directory. Just needed to change it to `/gitea-vet` to exclude only the binary.
jolheiser changed title from WIP: Add gitea-vet to Add gitea-vet 2 years ago
jolheiser removed the
status/wip
label 2 years ago
jolheiser approved these changes 2 years ago
Dismissed
appleboy reviewed 2 years ago
Dismissed
// Copyright 2018 The Gitea Authors. All rights reserved.
// Use of this source code is governed by a MIT-style
// license that can be found in the LICENSE file.
Collaborator

Why remove License header?

Why remove License header?
6543 commented 2 years ago
Poster
Collaborator

to test if gitea-vet works

the test SHOULD fail!!

to test if gitea-vet works the test SHOULD fail!!
6543 requested review from jolheiser 2 years ago
Poster
Collaborator

@jolheiser the test should fail! I removed a license header to test: see comments ☝️

@jolheiser the test should fail! I removed a license header to test: see comments :point_up:
Collaborator

Problem was from copy/paste to Makefile

Problem was from copy/paste to `Makefile`
jolheiser requested changes 2 years ago
Dismissed
jolheiser left a comment
Collaborator

issues.go is missing copyright. 😉

`issues.go` is missing copyright. :wink:
Poster
Collaborator

damed this cp issues - when I find them on other pull's its anoying but ... If I made them its eafen more anoying for me

damed this cp issues - when I find them on other pull's its anoying but ... If I made them its eafen more anoying for me
6543 requested review from jolheiser 2 years ago
Poster
Collaborator

@appleboy now this is ready ;)

@appleboy now this is ready ;)
6543 requested review from appleboy 2 years ago
jolheiser approved these changes 2 years ago
Dismissed
Poster
Collaborator

@appleboy your review is required

@appleboy your review is required
6543 added this to the v0.4.0 milestone 2 years ago
zeripath approved these changes 2 years ago
Dismissed
zeripath merged commit 4b059770ea into master 2 years ago
6543 deleted branch add-gitea-vet 2 years ago
continuous-integration/drone/pr Build is passing
The pull request has been merged as 4b059770ea.
Sign in to join this conversation.
Loading…
There is no content yet.