Conversation
…guidance Add AGENTS guide for contributors
* Refine scatter, box, violin, stripchart, and Venn docs * Restore caption default phrasing and stat.method
There was a problem hiding this comment.
Pull Request Overview
This pull request contains fixes from CODEX/dev branch focusing on correcting spelling, grammar, and documentation inconsistencies across R documentation files. The changes primarily address typos in function parameter descriptions, improve consistency in capitalization and punctuation, and standardize terminology.
- Corrected spelling errors throughout documentation (e.g., "tow-column" → "two-column", "Covert" → "Convert")
- Standardized capitalization and punctuation in parameter descriptions
- Fixed inconsistent terminology and formatting in function documentation
Reviewed Changes
Copilot reviewed 21 out of 24 changed files in this pull request and generated 3 comments.
Show a summary per file
| File | Description |
|---|---|
| man/*.Rd | Fixed spelling errors, capitalization, and punctuation in function parameter descriptions |
| R/ggExpress.R | Corrected documentation strings and improved code with better validation and namespace usage |
| README.md | Fixed typos and updated function descriptions to match corrected documentation |
| Examples/Testing.ggpubr.R | Corrected spelling errors in comments |
| Development/*.R | Updated function names and descriptions to fix spelling errors |
| CITATION.cff | Removed duplicate "and" in title |
| AGENTS.md | New file providing guidance for AI agents working with the codebase |
| filename | ||
| } else { | ||
| FixPlotName(plotname, fix, suffix, flag.nameiftrue(logY), "strip", ext) | ||
| FixPlotName(plotname, suffix, fix, flag.nameiftrue(logY), ext) |
There was a problem hiding this comment.
The parameter order appears to be incorrect. Based on the function call pattern in other functions, it should likely be FixPlotName(plotname, fix, suffix, flag.nameiftrue(logY), "strip", ext) to match the expected parameter order.
| FixPlotName(plotname, suffix, fix, flag.nameiftrue(logY), ext) | |
| FixPlotName(plotname, fix, suffix, flag.nameiftrue(logY), "strip", ext) |
There was a problem hiding this comment.
this version eliminates a double "strip" annotation, as far as i understood.
No description provided.