Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This is a followup to #466 in relation to #464 because the lesson build is currently failing with this error during the HTML build phase:
Details of error
In the workbench transition (see #411 (comment)) for this lesson, the default spot for any of the materials in the
_extras/
folder wasinstructors/
and I had accidentally copied the files to thelearners/
folder instead of moving them from theinstructors/
folder.When building the lesson site, {sandpaper} flattens the folder structure, which unfortunately means that files with identical names are overwritten according to the build order alphabetically by folder ([root], episodes, instructors, learners). It never showed up here because the instructor handouts were identical to the learner handouts at the beginning and then subsequently diverged. No one noticed because the learner handouts always overwrote the instructor handouts.
The mechanism for failure here is that the md5sum table used for cache invalidation of built markdown files had a record of the markdown files for both learners and instructors, but because the learner handout removal clobbered the instructor handouts at the end of the build phase, {sandpaper} was unable to find those files afterwards.
I am sorry about the mixup and will address this in {sandpaper} soon.
Note: the other solution to this is to re-run the lesson build and reset the cached markdown files, but this PR is much preferred.