-
Notifications
You must be signed in to change notification settings - Fork 6
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
Can we remove waldo dependency but still use it if it's there ? #7
Comments
Everything that you are doing to create types can all be done through Rcpp quite efficiently without any imports |
In the absolute sense I guess, though Rcpp will be the dependency then. But you'd have to reprogram {crayon} and {waldo} basically. Waldo in particular is quite amazing and does more than what my examples in Readme show. |
You could maybe just go with {diffobj} by @brodieG: https://github.com/brodieG/diffobj |
That's a good idea! {diffobj} depends only on {crayon}, which has no dependency. I think it will be a good compromise. |
Just an fyi, I have a long term plan to change the Aside, |
Thanks Brodie! |
It's nice for a package like {typed} to be dependy free, {waldo} has many dependencies itself, though all quite small I think.
{waldo} plays really well with this package though, so I want to use it. But we can use
if(requireNamespace("waldo"))
and move it to "Suggests" if we provide alternate errors.Let's have something stable first though.
The text was updated successfully, but these errors were encountered: