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.

Sign up for GitHub

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

Jump to bottom

minor suggested updates to package #298

Open

jreps opened this issue Aug 9, 2022 · 1 comment

Labels

good first issue

Collaborator

jreps commented Aug 9, 2022 •

edited

Loading

[fixed] add checkmate to check all inputs for PLP
use roxygen templates for common inputs to use standard terminologies (should we have OHDSI specific templates)?

jreps added good first commit good first issue and removed good first commit labels

Collaborator Author

jreps commented Aug 25, 2022

I fixed the check issues with using if (class(x) = 'y') so no longer need checkmate.

roxygen templates still needed

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment