Skip to content

142 line break before function

Created by: FlorenceMounier

fix: the use of line break(s), inline comments or commented line between function name and function() do not affect inflate() anymore

  • in /inst/tests-templates/dev-template-function-name-linebreak.Rmd

    • fix: add functions that present line break(s), inline comments or commented line between function name and function()
    • fix: add functions that present the "normal" syntax with no line break(s), inline comments or commented line between function name and function()
    • fix: for some functions with linebreaks, add examples with no @, with @export, with @nord because it changes the outputs of inflate()
    • fix: for some functions with linebreaks, and tests
  • in /tests/testthat/test-inflate-part2.R

    • fix, test: test if function names are read properly to build .R files in /R/*.R
    • fix, test: test if examples are inflated properly in /R/*.R and /man/*.rd
    • fix, test: test if tests are inflates properly in /tests
  • doc: update in /NEWS.md

in progress:

  • the examples are not in the right place in the .R files => fix parse_fun() in ~/fusen/R/inflate-utils.R
  • no valid test for /test => add one in ~/fusen/inst/tests-templates/dev-template-function-name-linebreak.Rmd

New strategy

  • parse_fun() is splited into several sub-functions with unit tests using examples of input (i.e. code)
    • 1 function to get the name of the function from the code
    • 1 function to get the example(s) (from roxygen in the chunk "function-" or in the chunk "example-") and place them in the right place in .R file (between last roxygen and function)
    • 1 function to add #' @noRd if necessary

Merge request reports

Loading