-
Notifications
You must be signed in to change notification settings - Fork 0
Standardize branding to lowercase **randomforge** across docs and #2
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
Changes from all commits
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change | ||||
|---|---|---|---|---|---|---|
|
|
@@ -16,9 +16,9 @@ knitr::opts_chunk$set( | |||||
| ) | ||||||
| ``` | ||||||
|
|
||||||
| # Introduction | ||||||
| ## Introduction | ||||||
|
|
||||||
| **RandomForge — An Open Project for Clinical Trial Randomization in R** | ||||||
| **randomforge — An Open Project for Clinical Trial Randomization in R** | ||||||
|
|
||||||
| `randomforge` is a newly started, open-source project on GitHub that aims to | ||||||
| provide a clean and transparent implementation of clinical trial randomization | ||||||
|
|
@@ -38,7 +38,7 @@ This vignette explains how to: | |||||
| 3. Use GitHub effectively even if you are new to it | ||||||
|
|
||||||
|
|
||||||
| # 1. Installing randomforge and running a simple randomization | ||||||
| ## Installing randomforge and running a simple randomization | ||||||
|
|
||||||
| At this stage, the package is not yet on CRAN. | ||||||
| You can install the development version directly from GitHub: | ||||||
|
|
@@ -78,8 +78,7 @@ blockSizes <- getBlockSizes(config$treatmentArmIds, 8) | |||||
|
|
||||||
| # Create a permuted block randomization method | ||||||
| randomMethodPBR <- getRandomMethodPBR( | ||||||
| blockSizes = blockSizes, | ||||||
| fixedBlockDesignEnabled = FALSE | ||||||
| blockSizes = blockSizes | ||||||
| ) | ||||||
|
|
||||||
| # Create a random allocation value service | ||||||
|
|
@@ -88,12 +87,12 @@ ravService$createNewRandomAllocationValues(config) | |||||
|
|
||||||
| # Create a few randomization results | ||||||
| resultList <- lapply(1:8, function(i) { | ||||||
| getNextRandomResult( | ||||||
| suppressMessages(getNextRandomResult( | ||||||
| randomDataBase = randomDataBase, | ||||||
| randomProject = randomProject, | ||||||
| randomMethod = randomMethodPBR, | ||||||
| randomAllocationValueService = ravService | ||||||
| ) | ||||||
| )) | ||||||
| }) | ||||||
|
|
||||||
| # Convert results to a data frame | ||||||
|
|
@@ -102,7 +101,7 @@ randomDataBase |> | |||||
| knitr::kable() | ||||||
| ``` | ||||||
|
|
||||||
| # 2. How to contribute to randomforge | ||||||
| ## How to contribute to randomforge | ||||||
|
|
||||||
| Contributions of all kinds are welcome: | ||||||
|
|
||||||
|
|
@@ -118,7 +117,7 @@ get involved. | |||||
| We are happy to support new contributors. | ||||||
|
|
||||||
|
|
||||||
| # 3. Option A: Get invited as a direct contributor | ||||||
| ### Option A: Get invited as a direct contributor | ||||||
|
|
||||||
| If you prefer not to deal with forks or pull requests, you can simply request | ||||||
| to be added as a contributor to the GitHub repository. | ||||||
|
|
@@ -137,24 +136,24 @@ Once added, you will be able to push branches and open pull requests directly | |||||
| in the main repository. | ||||||
|
|
||||||
|
|
||||||
| # 4. Option B: Contribute via Fork (recommended for most users) | ||||||
| ### Option B: Contribute via Fork (recommended for most users) | ||||||
|
||||||
| ### Option B: Contribute via Fork (recommended for most users) | |
| ## Option B: Contribute via Fork (recommended for most users) |
Copilot
AI
Nov 25, 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] Inconsistent branding formatting. Throughout the PR, "randomforge" is formatted in different ways:
- As italics
*randomforge*(line 36 in R/pkgname.R, line 218 here) - As bold
**randomforge**(lines 3, 12, 26, 65 in README.md, line 6 in NEWS.md) - As plain code
`randomforge`in most other places
For consistency across the codebase, establish a single formatting convention. Recommendation: Use bold **randomforge** when referring to the initiative/brand, and code formatting `randomforge` when referring to the R package.
| We appreciate your interest in contributing to the *randomforge* project. | |
| We appreciate your interest in contributing to the **randomforge** project. |
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] Inconsistent branding formatting. The branding is formatted as italics
*randomforge*here, but elsewhere in this PR it's formatted as bold**randomforge**(e.g., README.md line 12, NEWS.md line 6).For consistency, use the same formatting convention throughout. Recommendation: Use bold
**randomforge**when referring to the initiative/brand, and code formatting`randomforge`when referring to the R package.