Solving issue #35

[LuisLauM]

Solving issue #35 applying the solution #35 (comment)

[paleolimbot]

Thank you and sorry for taking so long to review!

I think that to pass CMD check this will need:

• Add #' @param include.chunks Use TRUE to include citations in code chunks to the documentation for bbt_detect_citations().
• Running devtools::document() when this project is open in RStudio

Technically include_chunks would be more consistent with current argument names than include.chunks; however, I'm happy to have this feature added either way!

[bwiernik]

@paleolimbot I can clean this up and merge if you like. I have some time today or this week.

[paleolimbot]

That would be great! It sounds like it would solve problems for quite a few people!

[dmi3kno]

I suggest we do it via options(), so that users would be able to adjust it. I am pushing a PR in few minutes. Let's take these two issues separately (in two different PRs)

[bwiernik]

@paleolimbot Can you add me as a collaborator again so I can merge this commit?

[paleolimbot]

Done!

[bwiernik]

@paleolimbot Sorry, apparently my internet connection wasn't working when I accepted the invitation and it didn't go through. Could you resend the collaboration invite? I've got the bugs in this PR all fixed.

[dmi3kno]

If you decide to merge this instead of #42 could you, please add more prefixes, other than just ^fig- and ^tbl-? I would ideally want a pair of eyes to look at my alternative implementation and critique it (#42).

[bwiernik]

@paleolimbot just bumping this

[LuisLauM]

@paleolimbot just bumping this

Oh, I am sorry. I did not realize that the changes on my fork will be taking into account here. I just have been using the original code of @paleolimbot but introducing some features that works for me. Again, my apologizes, I will remove this thread.