fix description #173
fix description #173
Conversation
There was a problem hiding this comment.
Pull request overview
This PR updates the R package DESCRIPTION file and refactors the CI/CD workflow. The primary changes involve formatting adjustments to package metadata and reorganizing the continuous integration build process.
Key Changes:
- Modified the package title capitalization in DESCRIPTION file
- Streamlined Authors@R field formatting by removing redundant Maintainer and Author fields
- Refactored CI workflow to check the built tarball instead of the source directory
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.
| File | Description |
|---|---|
| DESCRIPTION | Updated title capitalization and consolidated author information into Authors@R field only |
| .github/workflows/ci-dev.yaml | Removed vignette building step, reorganized R CMD check to run on built tarball with improved artifact handling |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| Package: NMAR | ||
| Type: Package | ||
| Title: Estimation under Not Missing at Random Nonresponse | ||
| Title: Estimation under not Missing at Random Nonresponse |
There was a problem hiding this comment.
The word "Not" should be capitalized in the package title to maintain proper capitalization of "Not Missing at Random" (NMAR), which is a standard statistical term. The original capitalization was correct.
| Title: Estimation under not Missing at Random Nonresponse | |
| Title: Estimation under Not Missing at Random Nonresponse |
| devtools::build_vignettes() | ||
| " | ||
| # |
There was a problem hiding this comment.
The commented-out line contains only a hash symbol with no explanation. If this is meant to be a placeholder or comment, it should either be removed entirely or include a meaningful comment explaining why the vignette building step was removed. Empty comments reduce code clarity.
| # |
No description provided.