Gábor Csárdi comments

Results 2480 comments of


                                            Gábor Csárdi

Passing multiline strings (e.g., `waldo::compare()` outputs) to `cli_abort`

Yes, cli re-wraps text to the screen width. If you are not happy about that, and as a current workaround, you can still use `rlang::abort()` directly, there is nothing wrong...

Passing multiline strings (e.g., `waldo::compare()` outputs) to `cli_abort`

This will be fixed in #460

`.url` style should probably use `style_hyperlink()`

It would be best to add hyperlink support to the `ansi_*()` functions as well.

`.url` style should probably use `style_hyperlink()`

I think we could think about this a bit more, so I'll skip it for now.

Encountering `callr_remote_error: subscript out of bounds` when loading remote package from GitHub

This is a bug in pak. Will fix soon, I'll let you know here. A workaround is to remove `Remotes` temporarily and install TreeTools manually after the rest is installed:...

Encountering `callr_remote_error: subscript out of bounds` when loading remote package from GitHub

Can you try my suggestion above? I.e. putting that step after the `setup-r-dependencies@v2` step? Does that not work? We do not support the `Additional_repositories` field currently, so it is ignored...

Encountering `callr_remote_error: subscript out of bounds` when loading remote package from GitHub

Even if you remove `Remotes`? (You might need to remove TreeDistData as well, or put that in `Remotes` instead of `ms609/TreeTools`.)

additional example for check-r-package with args

Would you like to submit a PR? (No pressure :)

additional example for check-r-package with args

There is an example in the README now.

Cancel in-progress jobs of the same workflow, in PRs

It is a bit more difficult, because there are no unique workflow ids, but nevertheless it is in #512. But not in v2 yet. In general it works great, with...