Closed brownsarahm closed 2 weeks ago
Thank you for your pull request :smiley:
:robot: This automated message can help you check the rendered files in your submission for clarity. If you have any questions, please feel free to open an issue in {sandpaper}.
If you have files that automatically render output (e.g. R Markdown), then you should check for the following:
:mag: Inspect the changes: https://github.com/carpentries/instructor-training/compare/md-outputs..md-outputs-PR-1643
The following changes were observed in the rendered markdown documents:
18-preparation.md | 12 ++++++++++--
md5sum.txt | 2 +-
2 files changed, 11 insertions(+), 3 deletions(-)
:stopwatch: Updated at 2024-06-04 15:56:46 +0000
That is a good point, actually with instructor view, maybe we do audience specific versions approximately like:
@brownsarahm what do you think of the changes I proposed in response to your suggestion?
I like it, thanks! @ndporter
attempts to fix #1642
I did something slightly different than the suggestion, but i agree there was a typo in the box.
I think leaving it at the top actually makes sense for two reasons:
If others 1) agree with that rationale 2) think it is worth making more findable, we could put a (shortend; edited for clarity) version of that justification in the box too.