Skip to content

Conversation

appleboy
Copy link
Member

@appleboy appleboy commented Jan 8, 2022

According to #18145 (comment) @wxiaoguang comment.

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

@appleboy appleboy added the type/docs This PR mainly updates/creates documentation label Jan 8, 2022
@codecov-commenter

This comment has been minimized.

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Jan 8, 2022
@wxiaoguang
Copy link
Contributor

Maybe the text should be replaced by make lint-backend, there is only something about lint-frontend but no information about lint-backend.

If I understand correctly, now make lint-backend can do all golangci-lint/vet (including revive/misspell), right?

@GiteaBot 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 8, 2022
Copy link
Member

@6543 6543 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

looking at it again - make lint-backend still is usefull!

@GiteaBot 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 8, 2022
@appleboy
Copy link
Member Author

appleboy commented Jan 8, 2022

@wxiaoguang Yes, golangci-lint include revive/misspell linter.

@6543
Copy link
Member

6543 commented Jan 8, 2022

I think we should still mention lint-backend

@lunny
Copy link
Member

lunny commented Jan 11, 2022

I think we should still mention lint-backend

That could be another PR.

@techknowlogick techknowlogick merged commit ee8718e into go-gitea:main Jan 11, 2022
zjjhot pushed a commit to zjjhot/gitea that referenced this pull request Jan 12, 2022
* 'main' of https://github.com/go-gitea/gitea:
  [doc] https-setup: explain relative paths for {CERT,KEY}_FILE fields. (go-gitea#18244)
  chore: remove unnecessary section (go-gitea#18209)
  Fix purple color in suggested label colors (go-gitea#18241)
  Prevent NPE when viewing non-rendered files (go-gitea#18234)
@wxiaoguang
Copy link
Contributor

I proposed a new PR

Hopefully we can keep documents correct and up-dated.

@appleboy appleboy deleted the docs branch January 12, 2022 15:07
Chianina pushed a commit to Chianina/gitea that referenced this pull request Mar 28, 2022
Signed-off-by: Bo-Yi Wu <[email protected]>

Co-authored-by: zeripath <[email protected]>
Co-authored-by: 6543 <[email protected]>
@go-gitea go-gitea locked and limited conversation to collaborators Apr 28, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. type/docs This PR mainly updates/creates documentation
Projects
None yet
Development

Successfully merging this pull request may close these issues.

8 participants