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.

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

Introduce lint-md and compliance-docs pipeline #24021

Merged
merged 5 commits into from
Apr 9, 2023
Merged

Conversation

silverwind
Copy link
Member

Alternative to #24017, see discussion there.

  • Add new Make target lint-md
  • Add new Drone pipeline compliance-docs
  • Add *.md to docs exclusion/inclusion
  • Consistently quote path.include and path.exclude statements in YAML

@GiteaBot GiteaBot added the lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. label Apr 9, 2023
@lunny lunny added the topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile label Apr 9, 2023
@lunny lunny added this to the 1.20.0 milestone Apr 9, 2023
@yardenshoham yardenshoham added lgtm/done This PR has enough approvals to get merged. There are no important open reservations anymore. and removed lgtm/need 2 This PR needs two approvals by maintainers to be considered for merging. labels Apr 9, 2023
@lunny lunny added the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Apr 9, 2023
@6543
Copy link
Member

6543 commented Apr 9, 2023

@silverwind sorry merged the pull that let the ci fail ... can we just push the fix here ... :/

@silverwind
Copy link
Member Author

silverwind commented Apr 9, 2023

I guess the next docs-only PR will verify this new pipeline then.

@silverwind
Copy link
Member Author

silverwind commented Apr 9, 2023

Hmm, did #24015 actually break the compliance CI again? If it did, we should get this in regardless of CI failure so that the following fix PR will fully validate.

@silverwind silverwind enabled auto-merge (squash) April 9, 2023 18:01
@6543 6543 disabled auto-merge April 9, 2023 18:04
@6543
Copy link
Member

6543 commented Apr 9, 2023

I'll force merge ...

@6543 6543 merged commit 96fb750 into go-gitea:main Apr 9, 2023
@6543
Copy link
Member

6543 commented Apr 9, 2023

Next doc pull must comply

@silverwind
Copy link
Member Author

#24024

@silverwind silverwind deleted the lint-md branch April 9, 2023 18:13
zjjhot added a commit to zjjhot/gitea that referenced this pull request Apr 10, 2023
* upstream/main: (39 commits)
  Fix protected branch for API (go-gitea#24013)
  [skip ci] Updated translations via Crowdin
  Fix markdownlint (go-gitea#24024)
  Introduce lint-md and compliance-docs pipeline (go-gitea#24021)
  Fix https setup doc zh-cn (go-gitea#24015)
  Replace tribute with text-expander-element for textarea (go-gitea#23985)
  Improve GetBoards and getDefaultBoard (go-gitea#22981)
  Expand/Collapse all changed files (go-gitea#23639)
  Show errors for KaTeX and mermaid on the preview tab (go-gitea#24009)
  Show protected branch rule names again (go-gitea#23907)
  Reference the `zh-CN` version of `reverse-proxies` in `https-support` (go-gitea#24016)
  Fix lint problem in `https-support.zh-cn.md` (go-gitea#24014)
  docs: HTTPS configuration for zh-cn (go-gitea#23039)
  Re-add initial wiki page text when editing the page (go-gitea#23984)
  [skip ci] Updated translations via Crowdin
  fix: do not escape space between PyPI repository url and package name… (go-gitea#23981)
  Make bindata static build parse builtin templates correctly (go-gitea#24003)
  Group template helper functions, remove `Printf`, improve template error messages (go-gitea#23982)
  Adjust sticky pr header to cover background (go-gitea#23956)
  Update github.com/google/go-github to v51 (go-gitea#23946)
  ...
@lunny lunny removed the reviewed/wait-merge This pull request is part of the merge queue. It will be merged soon. label Apr 10, 2023
@go-gitea go-gitea locked as resolved and limited conversation to collaborators Jul 31, 2023
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. topic/build PR changes how Gitea is built, i.e. regarding Docker or the Makefile
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants