Skip to content

Some refactoring around tests#275

Draft
maelle wants to merge 5 commits intomainfrom
snaps
Draft

Some refactoring around tests#275
maelle wants to merge 5 commits intomainfrom
snaps

Conversation

@maelle
Copy link
Copy Markdown
Contributor

@maelle maelle commented Jul 11, 2025

Not ready!

@maelle
Copy link
Copy Markdown
Contributor Author

maelle commented Jul 11, 2025

@mpadge I didn't even get to the OS-specific stuff. 🤪


h <- readLines (f)
# href
h[grep("Click to see", h)] <- "<p>Clickety clop</p>"
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

this and the package version thingie above is annoying, as I suspect it should be handled by the existing code?

For the href I think you were trying to very deterministically replace the temporary directory, this is more hacky.

</table>
<p>Click below for tallies of functions used in each package. Locations of each call within this package may be generated locally by running ‘s &lt;- pkgstats::pkgstats(<path to repo>)’, and examining the ‘external_calls’ table.</p>
<details>
<summary>base</summary>
Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

is that expected?

This was referenced Jul 11, 2025
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