-
Notifications
You must be signed in to change notification settings - Fork 10
feat: Add function to report bugs #279
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
Conversation
Merge branch 'main' of https://github.com/LAPKB/Pmetrics_rust # Conflicts: # R/PM_model.R
Merge branch 'main' of https://github.com/LAPKB/Pmetrics_rust # Conflicts: # R/PM_model.R
Merge branch 'main' of https://github.com/LAPKB/Pmetrics_rust # Conflicts: # R/PM_model.R
This reverts commit 4a3cad5.
The logs are now fully customizable, so changes to logs are no longer necesary to be propagated from pharmsol to PMcore to Pmetrics!
Deploys from actions, not a branch
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.
Pull Request Overview
This PR adds a new function PM_bug_report() to help users report bugs by collecting and displaying system information necessary for debugging. This addresses issue #13 by providing a streamlined way for users to gather diagnostic information when encountering problems with Pmetrics.
Key Changes:
- Added
PM_bug_report()function that collects OS, R, RStudio, package, Rust, and Cargo version information - Function includes optional clipboard copy functionality and provides links to GitHub issues
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| @@ -0,0 +1,121 @@ | |||
| #' @title Get Help and Report Issues | |||
| #' @description | |||
| #' `r lifecycle::badge("stable")` | |||
Copilot
AI
Nov 14, 2025
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.
The lifecycle badge is marked as 'stable', but this is a newly added function. Consider using 'experimental' or 'maturing' badge initially until the function has been tested in production use.
| #' `r lifecycle::badge("stable")` | |
| #' `r lifecycle::badge("experimental")` |
| output <- paste0( | ||
| "\n", | ||
| "====================================================================\n", | ||
| " PMETRICS HELP & BUG REPORT \n", |
Copilot
AI
Nov 14, 2025
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.
[nitpick] Corrected spelling of 'PMETRICS' to 'Pmetrics' to match the package name casing used throughout the codebase.
| " PMETRICS HELP & BUG REPORT \n", | |
| " Pmetrics HELP & BUG REPORT \n", |
Solves LAPKB#13