Skip to content
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

Updated setup_package() and added new_author() function #125

Merged
merged 60 commits into from
Feb 22, 2021
Merged

Conversation

henriquesposito
Copy link
Collaborator

@henriquesposito henriquesposito commented Feb 22, 2021

Description

These patch changes include updates to setup_package() function to accept more than two package authors and ORCID numbers and the addition of new_author() function that adds new package contributors to package description file.

Package

Collect

Correct

Connect

Checklist:

  • PR form
    • I have given this pull request an informative title
    • Description above itemizes changes under subtitles, e.g. "## Collection""
    • Any closed, fixed, or related issues are referenced and explained in the description above, e.g. "Fixed #0 by adding A"
    • Package builds on my OS without issues
  • PR checks all pass for latest commit
    • Package builds on Mac
    • Package builds on Windows
    • Package builds on Linux
    • CodeCov check: Package improves or maintains good test coverage
    • CodeFactor check: Package improves or maintains good style
  • Documentation
    • Any new or modified functions or data have roxygen style documentation in their .R scripts
    • Any longer functions are commented inline so that it is easier to debug in the future
    • PR description above and the NEWS.md file are aligned
    • DESCRIPTION file version is bumped by the appropriate increment (major, minor, patch)

BBieri and others added 30 commits February 10, 2021 10:15
Family and given were not properly defined for the second author
…packages()

Will be iterated on a bit later on
Still has a bug where one is unable to apply multiple elements to the comment vector. This is related to issue #91 of the desc package.
… function

Added a TODO for the bug mentionned in issue #91 of the desc package
@jhollway jhollway self-requested a review February 22, 2021 12:11
.Rbuildignore Show resolved Hide resolved
DESCRIPTION Show resolved Hide resolved
NEWS.md Outdated Show resolved Hide resolved
R/package_setup.R Show resolved Hide resolved
inst/templates/qPackage-DESC2.dcf Show resolved Hide resolved
inst/templates/qPackage-NEWS.md Outdated Show resolved Hide resolved
@henriquesposito henriquesposito merged commit 216b370 into main Feb 22, 2021
@henriquesposito henriquesposito changed the title Updated setup_packages() and added new_author() function Updated setup_package() and added new_author() function Mar 17, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment