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

simplify, parse documentation with roxygen2::parse_file() #142

Merged
merged 1 commit into from
May 1, 2024

Conversation

slager
Copy link
Contributor

@slager slager commented May 1, 2024

  • modify internal function .doc_parse()
    • remove some custom parsing code, and instead use roxygen2::parse_file() to parse documentation
  • remove internal function comment()
    • this was previously called by .doc_parse(), and is actually the source code of an un-exported roxygen2 function that had been copied into DataPackageR
  • no changes to UI or functionality
  • passes automated tests, and also tried it interactively with some real-world data packages

@slager slager changed the title simplify, parse with roxygen2::parse_file() simplify, parse documentation with roxygen2::parse_file() May 1, 2024
@slager slager merged commit b4c05db into develop May 1, 2024
7 checks passed
@slager slager deleted the roxy_block branch May 1, 2024 01:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant