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

ci: remove contrib/doxygen/ #20303

Merged
merged 1 commit into from
Sep 23, 2022
Merged

ci: remove contrib/doxygen/ #20303

merged 1 commit into from
Sep 23, 2022

Conversation

justinmk
Copy link
Member

Problem:

  • The layout of the doxygen HTML "dev docs" has been broken for 1+ years.
  • There is no evidence that anyone uses the doxygen HTML.
  • In the meantime since we introduced the doxygen HTML pages, the Nvim C docstrings have been surfaced in other ways:
    • Nvim LSP client
    • Generated :help docs like ":help api" and ":help ui".

Solution:

ref #824

Problem:
- The layout of the doxygen HTML "dev docs" has been broken for 1+ years.
- There is no evidence that anyone uses the doxygen HTML.
- In the meantime since we introduced the doxygen HTML pages, the Nvim
  C docstrings have been surfaced in other ways:
  - Nvim LSP client
  - Generated :help docs like ":help api" and ":help ui".

Solution:
- Remove `contrib/doxygen/` from neovim repo.
  - Note: keep `src/Doxyfile`, it is used by `scripts/gen_vimdoc.py`. ☠️
- Remove `doc/dev/` from website. neovim/neovim.github.io#291
- Remove doxygen job from the neovim/doc repo. neovim/doc#28
- Future/ongoing: ":help dev" and other :help docs will continue to be
  improved as the "source of truth" for developer-related docs.

ref neovim#824
@github-actions github-actions bot added the ci automation for build, test, and release label Sep 23, 2022
@justinmk justinmk merged commit 3fe4391 into neovim:master Sep 23, 2022
@justinmk justinmk deleted the rmdoxygen branch September 23, 2022 15:36
lvimuser pushed a commit to lvimuser/neovim that referenced this pull request Oct 6, 2022
Problem:
- The layout of the doxygen HTML "dev docs" has been broken for 1+ years.
- There is no evidence that anyone uses the doxygen HTML.
- In the meantime since we introduced the doxygen HTML pages, the Nvim
  C docstrings have been surfaced in other ways:
  - Nvim LSP client
  - Generated :help docs like ":help api" and ":help ui".

Solution:
- Remove `contrib/doxygen/` from neovim repo.
  - Note: keep `src/Doxyfile`, it is used by `scripts/gen_vimdoc.py`. ☠️
- Remove `doc/dev/` from website. neovim/neovim.github.io#291
- Remove doxygen job from the neovim/doc repo. neovim/doc#28
- Future/ongoing: ":help dev" and other :help docs will continue to be
  improved as the "source of truth" for developer-related docs.

ref neovim#824
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ci automation for build, test, and release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant