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

VignetteBuilder DECSRIPTION field split over lines is not recognized #439

Closed
jackwasey opened this issue Aug 17, 2018 · 3 comments
Closed

VignetteBuilder DECSRIPTION field split over lines is not recognized #439

jackwasey opened this issue Aug 17, 2018 · 3 comments

Comments

@jackwasey
Copy link

@jackwasey jackwasey commented Aug 17, 2018

I have a package DESCRIPTION which, after using desc package to reformat, contains:

VignetteBuilder:
    knitr

In this case, using

usethis:::use_description_field("VignetteBuilder", "knitr")

or, as I originally did:

use_vignette("Medical Twitter strategies used by other authors")

gives the error:

Error: VignetteBuilder has a different value in DESCRIPTION. Use overwrite = TRUE to overwrite.

Furthermore, the error message refers to an argument to usethis:::use_description_field which is not passed through by use_vignette and thus it is not possible to act on this by doing use_vignette("name", overwrite = TRUE)

@hadley
Copy link
Member

@hadley hadley commented Nov 25, 2018

@gaborcsardi there's no standard desc API for fields that are comma separated values, right? So this will require an issue in desc first.

@gaborcsardi
Copy link
Contributor

@gaborcsardi gaborcsardi commented Nov 25, 2018

@hadley yeah, but this is relatively simple, and there are not too many comma separated fields. So maybe usethis could just handle this?

@hadley
Copy link
Member

@hadley hadley commented Nov 26, 2018

Yeah, I can handle it too.

@hadley hadley closed this in efafb28 Nov 30, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
3 participants