-
Notifications
You must be signed in to change notification settings - Fork 1
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
create_pb_agent() does not match any columns if level > 1 #2
Comments
Thank you so much for taking the time to check out the package and report this issue! Now that you say it, I realize my tests only do cover the level-1 checks and are probably too biased towards my own person workflow. I really appreciate your pointing this out. If you wouldn't mind submitting your fix as a PR, that would be fantastic! Thanks again! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Hi Emily,
Excellent idea. I took the package for a spin - mainly to see if it can live with the not-recommended scheme of having the measure in the second level.
That's how I found a minor bug in
create_pb_agent()
, where no checks are run by {pointblank} iflevel
is set to 2 or more.Reprex:
# interrogate(cagent2) # shows no checks if run in an interactive session
Created on 2021-01-13 by the reprex package (v0.3.0)
Looking at the debug, I think it's caused by a bit of regex. I am aware this is experimental but in case it would be helpful I'd be happy to submit my local fix as a PR.
The text was updated successfully, but these errors were encountered: