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

potential fix to copy quizzes to manuscript dir #80

Merged
merged 2 commits into from
Sep 29, 2021
Merged

Conversation

carriewright11
Copy link
Member

Purpose/implementation Section

What changes are being implemented in this Pull Request?

Modifying the bookdown_to_leanpub code to copy quizzes

What was your approach?

#79

What GitHub issue does your pull request address?

Tell potential reviewers what kind of feedback you are soliciting.

Copy link
Contributor

@cansavvy cansavvy left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It looks good to me (assuming checks pass) Ignore windows check still. UGh #73

@@ -237,6 +244,10 @@ bookdown_to_leanpub <- function(path = ".",
message("Copying bib files")
}
copy_bib(path, output_dir = output_dir)
copy_quizzes(path, output_dir = output_dir)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just so its parallel to the others, can we put message first, then function?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

yup!

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yup that's it!

@carriewright11
Copy link
Member Author

Look good now? we can make an issue about adding a check for quizzes later

@cansavvy cansavvy merged commit 09123b6 into master Sep 29, 2021
@cansavvy
Copy link
Contributor

Look good now? we can make an issue about adding a check for quizzes later

The checking machinery exists: https://jhudatascience.org/leanbuild/docs/reference/check_quizzes.html
Just need to know what else I should add to it for checking: #52

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