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

When a Liquid Exception is caught, show the full path (rel to site source) #1415

Merged
merged 1 commit into from Aug 14, 2013

Conversation

Projects
None yet
4 participants
@parkr
Member

parkr commented Aug 12, 2013

Instead of just the basename of the page or post that has encountered a Liquid
exception, it will print the path of the item being processed, relative to the
site source.

Old:

  • docs/index.markdown has a problem
  • outputs Liquid Exception: ... index.markdown

New:

  • docs/index.markdown has a problem
  • outputs Liquid Exception: ... docs/index.markdown
@parkr

This comment has been minimized.

Member

parkr commented Aug 12, 2013

@benbalter this may affect what you are doing.

@mattr-

This comment has been minimized.

Member

mattr- commented Aug 14, 2013

👍 :shipit:

parkr added a commit that referenced this pull request Aug 14, 2013

Merge pull request #1415 from mojombo/full-path-output-for-liquid-exc…
…eption

When a Liquid Exception is caught, show the full path (rel to site source)

@parkr parkr merged commit 33ac651 into master Aug 14, 2013

1 check failed

default The Travis CI build failed
Details

@parkr parkr deleted the full-path-output-for-liquid-exception branch Aug 14, 2013

parkr added a commit that referenced this pull request Aug 14, 2013

@kelvinst

This comment has been minimized.

kelvinst commented Aug 14, 2013

I think with this, th PR #1266 and the issue #1111 can be both closed.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.