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

Added option of multiple messy bib files #80

Merged
merged 3 commits into from
Jan 5, 2021
Merged

Conversation

ilsePit
Copy link
Contributor

@ilsePit ilsePit commented Dec 30, 2020

Line 6: changed description messy_bibliography parameter
Line 23: changed to allow for messy_bibliography to be a list of characters
Line 54-57: changed to allow for multiple messy bib files to be used as input in the function tidy_bib_file(), similar to how multiple Rmd files are possible as input.

Line 6: changed description messy_bibliography parameter
Line 23: changed to allow for messy_bibliography to be a list of characters
Line 54-57: changed to allow for multiple messy bib files to be used as input in the function tidy_bib_file(), similar to how multiple Rmd files are possible as input.
@crsh
Copy link
Owner

crsh commented Jan 5, 2021

Again, thanks! This should be very useful. Same question as in the other PR: Does this work if multiple Rmd files are as c("file1.Rmd", "file2.Rmd") and list("file1.Rmd", "file2.Rmd")? If the latter does not work, it may make sense to slightly change the wording in the documentation to "One path or a vector of paths"?

changed wording in line 6 from 'list' to 'vector' to properly reflect the fact that the input for messy_bibliography should be in the form of either "messy_references.bib" or c("messy1.bib", "messy2.bib")
@ilsePit
Copy link
Contributor Author

ilsePit commented Jan 5, 2021

you're right, changed it! thanks!

@crsh
Copy link
Owner

crsh commented Jan 5, 2021

Thanks once more. Greatly appreciated!

@crsh crsh merged commit 3d5b19b into crsh:master Jan 5, 2021
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