Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

build docs on each PR #24890

Closed
wants to merge 3 commits into from
Closed

build docs on each PR #24890

wants to merge 3 commits into from

Conversation

techknowlogick
Copy link
Member

It is minimal extra CI time to build docs on each PR.

@techknowlogick techknowlogick added topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. labels May 23, 2023
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label May 23, 2023
@pull-request-size pull-request-size bot added the size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. label May 23, 2023
@silverwind
Copy link
Member

silverwind commented May 23, 2023 •
edited
Loading

Likely conflicted by #24877.

@techknowlogick @yardenshoham
Update .github/workflows/pull-compliance-docs.yml
61c4c17 
Co-authored-by: Yarden Shoham <git@yardenshoham.com>
@pull-request-size pull-request-size bot added size/S Denotes a PR that changes 10-29 lines, ignoring generated files. and removed size/XS Denotes a PR that changes 0-9 lines, ignoring generated files. labels May 24, 2023
@techknowlogick
Copy link
Member Author

Likely conflicted by #24877.

Yeah, this is a stopgap until that could be merged.

@silverwind
Copy link
Member

I think #24877 is ready for a merge and a few test runs, but happy to merge this as well if it takes longer.

@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 May 24, 2023
@techknowlogick techknowlogick deleted the techknowlogick-patch-1 branch May 25, 2023 02:01
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Aug 23, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@yardenshoham yardenshoham yardenshoham left review comments

@silverwind silverwind silverwind approved these changes

Assignees
No one assigned
Labels
lgtm/need 1 This PR needs approval from one additional maintainer to be merged. size/S Denotes a PR that changes 10-29 lines, ignoring generated files. skip-changelog This PR is irrelevant for the (next) changelog, for example bug fixes for unreleased features. topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@techknowlogick @silverwind @yardenshoham @GiteaBot