rmm icon indicating copy to clipboard operation
rmm copied to clipboard

[FEA] Disable doxygen documentation publication

Open vyasr opened this issue 1 year ago • 3 comments

Is your feature request related to a problem? Please describe. After #1324 the C++ APIs of rmm are documented within the Sphinx documentation along with the Python APIs. As a result, there is no longer a need for publication of the doxygen docs. However, they were left in place to allow easy comparison during the remainder of the 23.12 development cycle.

Describe the solution you'd like We should stop publishing the doxygen docs before the release.

vyasr avatar Nov 02 '23 22:11 vyasr

Linking this comment in particular that identify the necessary follow-up tasks

https://github.com/rapidsai/rmm/pull/1324#pullrequestreview-1710741054

ajschmidt8 avatar Nov 03 '23 21:11 ajschmidt8

Hope you don't mind, I'm assigning you @ajschmidt8 and @vyasr. Do you think it's reasonable to finish this in 24.02? We can push to 24.04 if not.

harrism avatar Jan 18 '24 11:01 harrism

Yup assigning us is fine. We've already stopped building the doxygen HTML, so I think we should be good to go for this release. Perhaps a website update is still needed though, @ajschmidt8 should I modify https://github.com/rapidsai/docs/pull/464 to just remove rmm and get that merged, then come back to cudf when we remove it?

vyasr avatar Jan 18 '24 20:01 vyasr

@ajschmidt8 @vyasr is this going to happen in 24.04 or should I move to 24.06?

harrism avatar Mar 06 '24 20:03 harrism

As far as I can tell, this was completed in the PRs below:

  • https://github.com/rapidsai/rmm/pull/1415
  • https://github.com/rapidsai/docs/pull/478

@vyasr, any remaining items here? or can we close it? I think that should be everything.

ajschmidt8 avatar Mar 12 '24 17:03 ajschmidt8

Confirmed with Vyas offline that it is complete. Closing.

ajschmidt8 avatar Mar 12 '24 17:03 ajschmidt8