modelsummary's coef_rename doens't work with logical variables - #1240
Conversation
There was a problem hiding this comment.
Code Review
This pull request adds support for logical parameters by updating parameter type detection, name cleaning, and formatting. The review feedback highlights a potential bug where globally stripping "TRUE$" from parameter names could corrupt non-logical parameters ending in "TRUE", suggesting a safer local handling instead. Additionally, an efficiency improvement is recommended to subset the character vector of names directly rather than subsetting the entire data frame when identifying logical columns.
Important
The consumer version of Gemini Code Assist on GitHub is being sunset. Starting June 18, 2026, new organization installations will be blocked, and all code review activity will officially cease on July 17, 2026.
For more details on the timeline and next steps, please review the Help Documentation.
Fixes #1239