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

Remove HTML builds of librmm #1415

Merged
merged 2 commits into from
Dec 18, 2023

Conversation

vyasr
Copy link
Contributor

@vyasr vyasr commented Dec 18, 2023

Description

We no longer require separate librmm doc builds since they are incorporated into the Sphinx build now.

Checklist

  • I am familiar with the Contributing Guidelines.
  • New or existing tests cover these changes.
  • The documentation is up to date with these changes.

@vyasr vyasr added 3 - Ready for review Ready for review by team doc Documentation breaking Breaking change labels Dec 18, 2023
@vyasr vyasr self-assigned this Dec 18, 2023
@github-actions github-actions bot added the cpp Pertains to C++ code label Dec 18, 2023
@vyasr vyasr requested a review from a team as a code owner December 18, 2023 21:24
@github-actions github-actions bot added the ci label Dec 18, 2023
Copy link
Member

@raydouglass raydouglass left a comment

Choose a reason for hiding this comment

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

This means that docs should be disabled for 24.02+ aka nightly here right?

Might apply to stable/23.12 too?

@vyasr
Copy link
Contributor Author

vyasr commented Dec 18, 2023

@raydouglass yup that PR is rapidsai/docs#464

@vyasr
Copy link
Contributor Author

vyasr commented Dec 18, 2023

/merge

@rapids-bot rapids-bot bot merged commit 77b5500 into rapidsai:branch-24.02 Dec 18, 2023
47 checks passed
@vyasr vyasr deleted the chore/disable_doxygen_html branch December 18, 2023 22:49
bdice added a commit to bdice/rmm that referenced this pull request Jul 10, 2024
rapids-bot bot pushed a commit that referenced this pull request Jul 15, 2024
This reverts commit 77b5500.

See: rapidsai/docs#517 (review)

We need to have librmm docs published in order to enable cross-linking of librmm from libcudf docs.

From discussion with @raydouglass, we don't need to link to them publicly on docs.rapids.ai, as they are already included in the rmm docs (both C++ and Python). The librmm docs will just be used to cross-reference from libcudf docs.

Authors:
  - Bradley Dice (https://github.com/bdice)

Approvers:
  - Ray Douglass (https://github.com/raydouglass)
  - Vyas Ramasubramani (https://github.com/vyasr)

URL: #1604
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
3 - Ready for review Ready for review by team breaking Breaking change ci cpp Pertains to C++ code doc Documentation
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

None yet

2 participants