-
Notifications
You must be signed in to change notification settings - Fork 19
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
Glossary #23
Conversation
👍 I'll have a look at this at lunch |
... [term](glossary://term) ... | ||
``` | ||
|
||
Only the link to the term must be URL encoded. For example, the term "reuse identifier" would be defined in the yaml as `reuse identifier` but linked to as `glossary://reuse%20identifier`. |
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.
does this lint for these constraints?
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.
Good point, do you think the renderer should warn on this, or would that be best left to the linter?
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.
depends on your opinion on the workflow, would you work with say guard
where it would lint on save? If so then a warning would be good
Cool, only nitpicks, nothing seems blocking |
|
Fixes #18.
This is a work-in-progress because I haven't had a chance to test it with Swift Playgrounds to make sure it works. It's a bit messy, might be a better way to do this in Ruby. I'll check out to see if it works next week.