-
Notifications
You must be signed in to change notification settings - Fork 0
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
Pavels work + unit tests #5
Conversation
Couldn't get R CMD Check to see the |
|
||
## What Works | ||
|
||
| `box::use()` | Code completion | Param completion | Tooltip help | As of version | Notes | | ||
|---------------------------|:-:|:-:|:-:|--------:|:-:| | ||
| `pkg[...]` | | | | | | | ||
| `pkg[...]` | ✓ | | | 0.0.0.9001 | | |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Here are a few comments on what else works (code completion only)
box::use(pkg)
also works after this change (see attached video)box::use(pkg[attach_list])
kinda works - it will load all functions from a package, not limited to the ones in theattach_list
.prefix/mod[attach_list]
also works - just the bare minimum code completion tho, which is based solely on text parsing: i canbox::use()
a non-existent function, so not sure if we should check it (see second video).
Screen.Recording.2024-06-06.at.07.46.19.mov
Screen.Recording.2024-06-06.at.07.51.54.mov
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@Gotfrid , the plans are:
box::use(pkg)
: To auto-complete onpkg$...
such asdplyr$
should showdplyr$across
, ordplyr$fi
should showdplyr$filter
.box::use(pkg[attach_list])
: to only auto-complete the attached functions
Same goes for path/modules
.
Then there are the aliases.
Description
test-completion.R
pkg[...]
function name completion.Definition of Done
R CMD check
, linter, unit tests, spelling)..Rd
files withroxygen2::roxygenise()
)