Repository navigation
Modernization: repair man-page mojibake, extend biocViews - #11
Merged
Merged
Conversation
- Repair UTF-8 transcoding damage ("??" artifacts) in 7 dataset man
pages: author names (Gyorffy), copyright marks, and statistical
symbols (p38alpha, TGF-beta1, P <= , x 10^k), using ASCII
equivalents.
- biocViews: add ExpressionData, OvarianCancerData, GEO.
Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
- Merge the "(continued)" NEWS section into the 1.52.0 heading so news() shows one block per version. - Add miRNAData (TCGA.mirna.8x15kv2_eset) and ArrayExpress (E.MTAB.386_eset) biocViews; fix a missing comma space predating the mojibake. Co-Authored-By: Claude Fable 5 <noreply@anthropic.com>
There was a problem hiding this comment.
Copilot review overview
🟢 Approval recommended
The documentation and metadata updates are internally consistent and introduce no unresolved issues.
Review effort: Balanced
Findings: None
What changed in this PR
Repairs dataset documentation encoding artifacts and expands Bioconductor metadata discoverability.
Changes:
- Replaces mojibake with ASCII equivalents across seven dataset pages.
- Adds relevant
biocViewsterms and bumps the package version. - Consolidates NEWS entries and fixes author-list spacing.
| File | Description |
|---|---|
DESCRIPTION |
Bumps version and extends biocViews. |
NEWS.md |
Documents metadata and encoding repairs. |
man/GSE51088_eset.Rd |
Repairs copyright text. |
man/GSE49997_eset.Rd |
Repairs statistical and copyright symbols. |
man/GSE44104_eset.Rd |
Repairs TGF-beta1 notation. |
man/GSE32063_eset.Rd |
Repairs scientific notation. |
man/GSE32062.GPL6480_eset.Rd |
Repairs scientific notation. |
man/GSE30161_eset.Rd |
Fixes author-list spacing. |
man/GSE26193_eset.Rd |
Repairs p38alpha notation. |
man/GSE14764_eset.Rd |
Repairs an author name. |
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Follow-up pass deferred from #10 (not release-critical), mirroring waldronlab/curatedCRCData#6:
news()shows one block per version; fix a pre-existing missing comma space.R CMD checkStatus: OK.🤖 Generated with Claude Code