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

Fixed namespace issue due to roxygenise call #29

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ZacharyWaller
Copy link

Fixes #28

Since {roxygen2} 7.0.0 (released 2019/11/12) roxygenise uses pkgload::load_all() by default. This ends up loading up the devtools_shims namespace - which includes its own version of the ? function.

This means that when you load up {docstring} you can use docstring's version of ? but, behind the scenes, it runs roxygenise() which then loads up the devtools_shims namespace. Next time you use ? you're running the one loaded up in devtools_shims. This is the source of the error found.

Adding the load = "source" argument in the roxygenise() call bypasses loading up the devtools_shims namespace but still produces the desired behaviour.

@lukakoning
Copy link

@Dasonk Can you merge this PR? Available version is currently broken and we have to install this PR

@Dasonk
Copy link
Owner

Dasonk commented Nov 1, 2024

@Dasonk Can you merge this PR? Available version is currently broken and we have to install this PR

I'll look into it when I get home from work. A few years late isn't so bad right?

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.

The first function is documented but the following is not found
4 participants