Skip to content

Render theme-gem root only in development#9680

Merged
jekyllbot merged 2 commits intojekyll:masterfrom
ashmaroli:theme-drop-development-root
Sep 16, 2024
Merged

Render theme-gem root only in development#9680
jekyllbot merged 2 commits intojekyll:masterfrom
ashmaroli:theme-drop-development-root

Conversation

@ashmaroli
Copy link
Copy Markdown
Member

  • This is a 🐛 bug fix.
  • I've adjusted the documentation
  • The test suite passes locally

Summary

{{ theme.root }} rendering the absolute path to theme-gem has no value outside development builds.
So, render the value only when site is built in development mode explicitly.

NOT A BREAKING CHANGE

Consider backport to 4.3-stable branch.

@ashmaroli
Copy link
Copy Markdown
Member Author

@jekyllbot: merge +fix

@jekyllbot jekyllbot merged commit 32074ef into jekyll:master Sep 16, 2024
jekyllbot added a commit that referenced this pull request Sep 16, 2024
@ashmaroli ashmaroli deleted the theme-drop-development-root branch September 16, 2024 15:41
github-actions bot pushed a commit that referenced this pull request Sep 16, 2024
Ashwin Maroli: Render theme-gem root only in development (#9680)

Merge pull request 9680
ashmaroli added a commit to ashmaroli/jekyll that referenced this pull request Sep 16, 2024
@ashmaroli ashmaroli added the backport-candidate Consider for merge into an older stable branch label Sep 16, 2024
ashmaroli added a commit that referenced this pull request Sep 16, 2024
@jekyll jekyll locked and limited conversation to collaborators Sep 16, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

backport-candidate Consider for merge into an older stable branch bug fix frozen-due-to-age

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants