Skip to content

Add Jenna Jia Li to package author list - #137

Merged
matt-graham merged 4 commits into
mainfrom
mmg/add-jennajiali-as-author
Jul 23, 2026
Merged

matt-graham merged 4 commits into
mainfrom
mmg/add-jennajiali-as-author

Conversation

@matt-graham

@matt-graham matt-graham commented Jul 22, 2026 •

Copy link
Copy Markdown
Collaborator

Adds @jennajiali to package author list in DESCRIPTION file.

@jennajiali I have listed your name currently to match what you have on your GitHub profile, treating both 'Jenna' and 'Jia' as given names, but if you would like it formatted differently let me know. The first argument to person is meant to be a given name or names, and the second argument a family name.

The details here are fed in to the CRAN metadata for the package when we make a release and in the automatically generated citation for the package. If you happen to already have or would like to create an ORCiD (a free, unique persistent identifier for researchers which can be linked to various research outputs, publications etc. https://orcid.org/) we can also add that to the metadata.

@matt-graham matt-graham added the documentation Improvements or additions to documentation label Jul 22, 2026
@codecov

codecov Bot commented Jul 22, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (f399737) to head (4a0cc05).

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #137   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           11        11           
  Lines          762       762           
=========================================
  Hits           762       762           

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@jennajiali

Copy link
Copy Markdown
Contributor

Many thanks, @matt-graham!

I would prefer to format my name as c("Jenna", "Jia”) to represent "Jenna" and "Jia" as two distinct given names, rather than bundling them into a single string. This also matches the way your given names are formatted in the file.

I have also just created an ORCID account: https://orcid.org/0009-0007-7364-6389

@matt-graham

Copy link
Copy Markdown
Collaborator Author

Many thanks, @matt-graham!

I would prefer to format my name as c("Jenna", "Jia”) to represent "Jenna" and "Jia" as two distinct given names, rather than bundling them into a single string. This also matches the way your given names are formatted in the file.

Thanks for catching this, the names definitely should be separated in to different vector elements, that was a mistake on my part.

I have also just created an ORCID account: https://orcid.org/0009-0007-7364-6389

Now added this

@matt-graham
matt-graham merged commit eb90251 into main Jul 23, 2026
10 checks passed
@matt-graham
matt-graham deleted the mmg/add-jennajiali-as-author branch July 23, 2026 09:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants