-
-
Notifications
You must be signed in to change notification settings - Fork 122
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
'code' folder should be re-established/replaced #286
Comments
I would propose that we recreate a That said, I quite like the idea of including the code for plots as inline Instructor Notes (where that code is not part of the lesson content). If we do take that approach, we should make sure those notes have headings that clearly indicate what they contain. |
I like the latter idea. It will encourage us to maintain parity with the figures. I guess details on how to make an inline instructor note are in the workbench docs. |
Yes, here: https://carpentries.github.io/sandpaper-docs/episodes.html#instructor-notes You might also like this helpful video, featuring a lesson repository you may be familiar with 😉 |
The code folder was removed in #222, before the transition. My 2 cents: if the code directory is intended to generate figures for the lesson by maintainers, but not instructors or learners, then it's best to create it at the top level and it will not be included in the rendered lesson site. if, however you want to include the code as part of the repository, then it should live in the |
I investigated this a bit and it doesn't seem like the code in #222 is the full code for generating all the figures in the lesson. Is it possible that code lived elsewhere or got removed in a different PR? |
@bobturneruk in #221:
The text was updated successfully, but these errors were encountered: