Handle file:// values containing leading/trailing spaces #640

Merged
merged 1 commit into from Feb 7, 2014

Conversation

Projects
None yet
2 participants
@jamesls
Member

jamesls commented Feb 7, 2014

Fixes #631. The issue was that shorthand processing was triggered
because we were looking at leading spaces instead of the first non
space character.

Handle file:// values containing leading/trailing spaces
Fixes #631.  The issue was that shorthand processing was triggered
because we were looking at leading spaces instead of the first non
space character.
@toastdriven

This comment has been minimized.

Show comment
Hide comment
@toastdriven

toastdriven Feb 7, 2014

Contributor

LGTM. :shipit:

Contributor

toastdriven commented Feb 7, 2014

LGTM. :shipit:

@jamesls jamesls merged commit 0fca1a8 into aws:develop Feb 7, 2014

1 check passed

default The Travis CI build passed
Details

@jamesls jamesls deleted the jamesls:json-spaces branch Jun 23, 2014

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