-
Notifications
You must be signed in to change notification settings - Fork 1
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
CRAN release #8
Comments
Hi @nevrome, nice to hear from you. I agree that we should go ahead and release rdoxygen 2.0.0 on CRAN. I'm happy to take over maintenance of the package, thanks for offering. I'll admit that, while I still think that C++ documentation in R packages is a critical element error-free, portable, and efficient C++ code, I'm a bit less excited about doxygen vignettes. The main reason is that doxygen HTML documentation is rather large, adding ~300Kb as an upfront cost (although the extra Kb per documentation line is very little). All this to say that I've yet to include doxygen documentation in any of my recent CRAN releases (although the doxygen markup is still there :). I'm working on a satisfactory solution to this, but I think the quickest approach would be to provide a simple mechanism to include the doxydoc in an automatically-generated pkgdown website. Either way, I'm very glad to stay in touch :) |
OK - excellent. pkgdown might be a great extension for the future! For the procedure: We somehow have to change the maintainership on CRAN. I suggest we try to do this as follows:
|
Oh - You have to get rid of your fork. I can't transfer the repo on github if mlysy/rdoxygen already exists. |
Thanks for looking into this. mlysy/rdoxygen has been deleted. |
Is it not easy to simple exclude the generated output using .Rbuildignore? |
Unfortunately not without excluding the vignette altogether. That is, when you obtain a package from CRAN the vignettes do not get built. They are pre-built and shipped inside |
Yes, that is what I meant -- exclude the vignette altogether. I think it
will still work with pkgdown on github but you would not include the
vignette in CRAN (although I have not tested that setup).
…On Wed, Feb 26, 2020 at 9:18 AM mlysy ***@***.***> wrote:
Unfortunately not without excluding the vignette altogether. That is, when
you obtain a package from CRAN the vignettes do not get built. They are
pre-built and shipped inside inst/doc.
—
You are receiving this because you commented.
Reply to this email directly, view it on GitHub
<#8?email_source=notifications&email_token=AAEQXSIEQJ3ZBANLZFPUWIDRE2B5HA5CNFSM4HVHVAJKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOENAUGJY#issuecomment-591479591>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AAEQXSJ6SOIXAXG5T3H5K6DRE2B5HANCNFSM4HVHVAJA>
.
|
@mlysy
Sorry for disappearing for 10+ weeks. I think we can release this new version 2.0.0 of rdoxygen to CRAN. I updated the
NEWS.md
and thecran-comments.md
.Before we do so: Do you want to become the main maintainer of this package? I realized that I did not work with Rcpp for a long time now and I also believe you understand this package better than I do. I would love to stay in touch but pass the main responsibility on to you.
The text was updated successfully, but these errors were encountered: