Add clarity to docs on permalinks placeholders and builtins#8995
Add clarity to docs on permalinks placeholders and builtins#8995jekyllbot merged 7 commits intojekyll:masterfrom
Conversation
- global permalink doesn't have "output_ext" placeholder as documentation above tells - wrong "W:" removed
|
This is my attempt to write regex for:
|
|
I went to go double check the code and now I'm confused. The documentation mirrors the code exactly. When you use these permalink styles on a site, are you seeing different results? |
🤔 |
ashmaroli
left a comment
There was a problem hiding this comment.
Blocking this since docs should mirror Ruby codebase.
|
Agreed. But it is mentioned under subsequent section towards end of page
Agreed that it has not been documented anywhere. Perhaps this pull request could add that info.. |
What placeholder prefixes are allowed? |
@EmilySeville7cfg See underlying code for reference: jekyll/lib/jekyll/configuration.rb Lines 272 to 278 in 19868d5 Built-ins are syntactic-sugars (or shorthands) to a particular implementation. P.S. You may add a note to reduce confusion.. Also, please update the PR title and description to better describe latest diff. |
Co-authored-by: Ashwin Maroli <ashmaroli@users.noreply.github.com>
|
Thanks @EmilySeville7cfg |
Emily Grace Seville: Add clarity to docs on permalinks placeholders and builtins (#8995) Merge pull request 8995
Summary
:output_extas a placeholderWprefix to:weekwhen usingweekdatebuiltinContext
Questions
What to do with
dateandprettyformats now? I want fix documentation errors because it's hard to write JSON schema when they are exist.