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.

Sign up for GitHub

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

Jump to bottom

Sourcemaps should not be rendered by Liquid #123

Merged
merged 1 commit into from
Sep 17, 2021
Merged

Sourcemaps should not be rendered by Liquid #123

merged 1 commit into from
Sep 17, 2021

Conversation

ashmaroli
Copy link
Member

Summary

Resolves issue brought to light by @captn3m0 via failing test in #121
Closes #121

@ashmaroli ashmaroli added the fix label Aug 30, 2021
@ashmaroli ashmaroli requested a review from DirtyF August 30, 2021 14:27
@ashmaroli ashmaroli requested review from mattr- and removed request for DirtyF September 17, 2021 05:22
@mattr-
Copy link
Member

mattr- commented Sep 17, 2021

@jekyllbot: merge +fix

@jekyllbot jekyllbot merged commit 30361c6 into jekyll:master Sep 17, 2021
@jekyllbot jekyllbot added the bug label Sep 17, 2021
jekyllbot added a commit that referenced this pull request Sep 17, 2021
@jekyllbot
Update history to reflect merge of #123 [ci skip]
1d7d928
@ashmaroli ashmaroli deleted the sourcemap-no-render-liquid branch September 17, 2021 14:25
@mattxwang mattxwang mentioned this pull request Mar 8, 2022
Merged
@jekyll jekyll locked and limited conversation to collaborators Sep 17, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers

@mattr- mattr- mattr- approved these changes

Assignees
No one assigned
Labels
bug fix frozen-due-to-age
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@ashmaroli @mattr- @jekyllbot