-
Notifications
You must be signed in to change notification settings - Fork 8
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
Tidy up summary
and as_gt
functions
#282
Labels
best-practice
Enhance programming best practice
Comments
27 tasks
Hi @jdblischak , considering we have couple of PRs to improve |
Or I can create a PR to clean up the summary functions, and close this issue once that PR is merged. |
Sounds great, thank you, @yihui ! |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
The summary and as_gt functions in their current state are extremely lengthy. A preferable approach would be to condense them into a shorter function that utilizes sub-functions where necessary.
The text was updated successfully, but these errors were encountered: