Skip to content
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

meta/refactor document lessons library #9

Merged
merged 4 commits into from
Feb 10, 2024

Conversation

djacu
Copy link
Owner

@djacu djacu commented Feb 10, 2024

  • Add doc-comment for getFileExtension.
  • Rename findStrings to multilineMatch and add doc-comment.
  • Create makeFencedCodeBlock expression.
  • Refactor createLessonMetadata to accept an attrset for high level arguments.
  • Refactor createLessonMetadata to use more of the new functions.
  • Refactor createLessonMetadata by moving keys that don't need to be in the output to the let/in block.
  • Refactor filesToSubstitute to append the parent path to the file name.
  • Refactor textToSubstitute to use the new makeFencedCodeBlock expression.
  • Add doc-comment for makeFencedCodeBlock.
  • Add comments for all the higher level functions that create metadata or building markdown documentation.
  • Add doc-comment for getLessons.

djacu added 4 commits February 9, 2024 20:51
Rename findStrings to multilineMatch and add doc-comment.
Create makeFencedCodeBlock expression.
Refactor createLessonMetadata to accept an attrset for high level arguments.
Refactor createLessonMetadata to use more of the new functions.
Refactor createLessonMetadata by moving keys that don't need to be in the output to the let/in block.
Refactor filesToSubstitute to append the parent path to the file name.
Refactor textToSubstitute to use the new makeFencedCodeBlock expression.
Copy link

@github-actions github-actions bot temporarily deployed to pull request February 10, 2024 05:20 Inactive
@djacu djacu merged commit 3a39005 into main Feb 10, 2024
2 checks passed
@djacu djacu deleted the meta/refactor-document-lessons-library branch February 10, 2024 05:22
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.

1 participant