Skip to content

Conversation

TylerZeroMaster
Copy link
Contributor

https://openstax.atlassian.net/browse/CORE-1161

Add basic recipe for super documents. This is still kind of a skeleton, but it serves as a MVP that can be built upon.

Note: the original idea was to exclude the ToC in baking, however, later parts of the pipeline depend on the baked ToC. Instead of excluding it here, it will continue to be removed in the step-prepare-ancillaries step.

Removes empty answer key composite chapters
Skip add_title in bake_learning_objectives if title exists when this is set
Will likely need a lot more, but chapter review came up already
Copy link

codecov bot commented Sep 4, 2025

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (beaff0f) to head (526583e).
⚠️ Report is 1 commits behind head on main.

Impacted file tree graph

@@            Coverage Diff            @@
##              main      #418   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files          219       219           
  Lines         3681      3687    +6     
=========================================
+ Hits          3681      3687    +6     
Files with missing lines Coverage Δ
lib/kitchen/directions/bake_learning_objectives.rb 100.00% <100.00%> (ø)
...move_solutions_to_answer_key/answer_key_cleaner.rb 100.00% <100.00%> (ø)
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

Copy link

@RoyEJohnson RoyEJohnson left a comment

Choose a reason for hiding this comment

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

This looks reasonable to me, but I'm not familiar with this repo. You might want to have someone else look it over, as well.

@TylerZeroMaster TylerZeroMaster merged commit 0f8bb9c into main Sep 9, 2025
6 checks passed
@TylerZeroMaster TylerZeroMaster deleted the core-1161-super-documents branch September 9, 2025 19:08
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