Skip to content

Conversation

@rvedotrc
Copy link
Contributor

@rvedotrc rvedotrc commented Oct 16, 2025

Ideally #208 is merged before this one, so as to keep the two PRs separate.

  • chore: extract updateFileSection
  • fix: change marker name to match npm script name
  • feat: have 'generate:learning-goals' update the target file

@rvedotrc rvedotrc requested a review from a team as a code owner October 16, 2025 14:19
@rvedotrc rvedotrc changed the title write the goals feat: write the goals Oct 16, 2025
Base automatically changed from learning-dash-goals to main October 27, 2025 10:19
@@ -0,0 +1,39 @@
import * as fs from "node:fs/promises";
Copy link
Contributor

Choose a reason for hiding this comment

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

Awesome helper, thanks :)

@adamblanchard adamblanchard merged commit 2e5cc9b into main Oct 27, 2025
1 check passed
@adamblanchard adamblanchard deleted the write-the-goals branch October 27, 2025 10:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: No status

Development

Successfully merging this pull request may close these issues.

3 participants