Skip to content
This repository was archived by the owner on Mar 22, 2020. It is now read-only.

Added option 'allownull'#5

Open
tphipps wants to merge 1 commit into
predicthq:masterfrom
tphipps:option-allownull
Open

Added option 'allownull'#5
tphipps wants to merge 1 commit into
predicthq:masterfrom
tphipps:option-allownull

Conversation

@tphipps
Copy link
Copy Markdown
Contributor

@tphipps tphipps commented Oct 8, 2019

If the template resolves to an empty string, the current code simply concatenates all the parts of the address array together and returns that. This pull request adds an option "allownull" that allows a null string to be returned if the template resolves to that rather than the concatenation. Default behaviour is unchanged.

@coveralls
Copy link
Copy Markdown

coveralls commented Oct 8, 2019

Coverage Status

Coverage decreased (-0.3%) to 95.029% when pulling 561122c on tphipps:option-allownull into 5ed47ba on predicthq:master.

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

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants