-
Notifications
You must be signed in to change notification settings - Fork 12
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
Rework cookbook #149
Rework cookbook #149
Conversation
Once this is merged, the instructions in #129 will need to be updated a bit and the cookbook_skeletons branch updated to pre-enter the likely snippet names for convenience |
…ge_id` properties added to the helpers
…e anything that inherits from this can have a `label` property
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It looks great! Immense work. I'm just curious, why not consider test_cookbook an integration test and run it using unittest?
pymdownx.snippets
to load compile python code into the markdownutils/
that checks the validity of the cookbook recipes.label
properties (Allow AutoLabel to work in other places that have a .label property #155)datetime
etc #151 and Fix skeleton modifier bug + add ManifestRef changes #154 due to basing/branching fun. They should be merged first, and if either is changed before merge, there might be conflicts.Closes #155