Skip to content

Correct package author metadata for 0.2.2 - #255

Merged
jakeberv merged 1 commit into
mainfrom
codex/remove-clavel-package-author
Oct 17, 2026
Merged

jakeberv merged 1 commit into
mainfrom
codex/remove-clavel-package-author

Conversation

@jakeberv

@jakeberv jakeberv commented Oct 17, 2026 •

Copy link
Copy Markdown
Owner

Remove Julien Clavel's contributor/copyright-holder entry from DESCRIPTION's Authors@R, so it no longer appears in the generated package Author field or pkgdown's detailed contributor list. The entry did not carry the aut role. Attribution for the adapted mvMORPH initialization code remains in R/mvgls-starts.R; the mvMORPH paper citations are unchanged.

The maintainer confirmed this choice after reviewing the distinction between authorship and contributor/copyright credit. No package code or tests changed.

Validation on source commit 2f7b5a528757e281eca56c272d9b11aff7be919b:

  • Fresh isolated build and R CMD check --as-cran --run-donttest, including PDF and HTML manuals: 0 errors, 0 warnings, 1 NOTE (unable to verify current time).
  • 2,980 passing assertions, zero test warnings/failures, 138 expected skips; spelling and examples passed.
  • The 101-page PDF passed qpdf --check; temporary-directory detritus check passed.
  • Existing dependencies were accessed read-only, with unchanged DESCRIPTION hashes. At most two workers and one numerical thread per worker.
  • The rebuilt archive's Author field contains the ten intended bifrost authors. Only DESCRIPTION differs from the prior PR Add the Zenodo concept DOI to repository citation metadata #254 checked archive; source attribution and citations are unchanged.

Archive: output/release-0.2.2-author-check/run-wbvv4gjx/bifrost_0.2.2.tar.gz

SHA-256: 9800f576b5ba807e8ffa98da55a87c2a767dcfccd4db6be59a70c8dcaa1eed7a

This archive supersedes the previous submission archive. Evidence is recorded in tools/releases/0.2.2.md; that file and cran-comments.md are excluded from source archives, so the evidence update does not change the checked package contents. PR CI is separate from this completed local check.

Issue #231 remains deferred. This PR does not submit to CRAN or publish a release.

Copilot AI balanced review requested due to automatic review settings October 17, 2026 06:04
@jakeberv
jakeberv merged commit ff80558 into main Oct 17, 2026
12 checks passed
@jakeberv
jakeberv deleted the codex/remove-clavel-package-author branch October 17, 2026 06:05

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

🟢 Approval recommended

The isolated metadata change matches the stated scope and has no identified blocking issues.

No open issues 🎉

What changed in this PR

Corrects package author metadata for version 0.2.2 without changing runtime behavior.

Changes:

  • Removes Julien Clavel from Authors@R, retaining source attribution and mvMORPH citations.
File Description
DESCRIPTION Removes the contributor entry from package metadata.

Review effort: Balanced

@codecov

codecov Bot commented Oct 17, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants