Skip to content

Fix compile docs #24866

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
Closed
wants to merge 1 commit into from
Closed

Fix compile docs #24866

wants to merge 1 commit into from

Conversation

lunny
Copy link
Member

@lunny lunny commented May 22, 2023 •
edited
Loading

Fix #24530 (comment)

This PR fix a bug that when a PR includes docs, the compile docs workflow will not run.

@lunny lunny added the topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile label May 22, 2023
@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label May 22, 2023
@silverwind
Copy link
Member

As mentioned in #24530 (comment), there is too much wrong with the changes in #24530, I would fully revert it.

@lunny
Copy link
Member Author

lunny commented May 23, 2023

replaced by #24877

@lunny lunny closed this May 23, 2023
@lunny lunny deleted the lunny/fix_docs branch May 23, 2023 10:15
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Aug 21, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@techknowlogick techknowlogick Awaiting requested review from techknowlogick

@silverwind silverwind Awaiting requested review from silverwind

Assignees
No one assigned
Labels
lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. 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.
3 participants
@lunny @silverwind @GiteaBot