Export of non-exported functions to enable other style guides to pass CRAN checks

[Robinlovelace]

Please consider exporting the following functions:

• [ ] is_function_call()
• [ ] previous_non_comment()
• [ ] scope_normalize()

Context: I'm working on {styler.equals} based on @lorenzwalthert's very handy template: https://github.com/lorenzwalthert/styler.yours

As mentioned in #860 tests do not pass due to these lines:

if (styler:::is_function_call(pd)) {

https://github.com/lorenzwalthert/styler.yours/blob/07b6689532aab92b1c60f106f9daf0cbb1fed6e3/R/rules-line-break.R#L14

https://github.com/lorenzwalthert/styler.yours/blob/ea4eb50e79886c5bf58dec51dff76aafbcdd4249/R/core.R#L21

@MichaelChirico advocated in https://github.com/r-lib/styler/issues/340#issuecomment-1207569513 to export those functions to make it easier to for people building alternative styles building on this great package to release them to the masses.

[ShixiangWang]

Thanks @Robinlovelace

[Robinlovelace]

Thank you for instigating this Shixiang, hopefully many people will benefit :rocket:

[lorenzwalthert]

I am a bit hesitant to export these functions. First of all, they don't provide value for 99% of the users of {styler} and hence convolute the namespace, add aditional constraint on the development of {styler} (compatibility with downstream users). Another option is that you define these functions yourself (just c/p from styler) since they are relatively easy. Also, you needed a new {styler} release whenever you wanted to use a new {styler} internal (like now). Anyways, I guess it's the proper way to export them so I am open to PRs (unless we want to create more overhead with {styler.infra} package). Note that you need to wait for a {styler} release for your package to be accepted to CRAN. Meanwhile, you could try this workaround

is_function_call <- utils::getFromNamespace("is_function_call", "styler")

[Robinlovelace]

It sounds like there are 2 main options:

• PR to the template {styler.yours} package with copy-pasted versions of the functions
• PR to styler package exporting the functions

As you say both have pros and cons, which works best for you? Happy to try to put in a PR and mild preference for the former option but happy with either and there may be other good options.

[lorenzwalthert]

A PR to {styler} is fine. You can hence remove the @keywords internal and add a new {pkgdown} section in the references for these types of functions.

[lorenzwalthert]

@Robinlovelace if you want this feature, please send a PR.

[Robinlovelace]

@Robinlovelace if you want this feature, please send a PR.

Thanks for the nudge @lorenzwalthert and see my attempts at implementing this here: https://github.com/r-lib/styler/pull/1037

Comments and changes welcome :pray: