Skip to content
This repository has been archived by the owner on Aug 23, 2018. It is now read-only.

Allow document to be undefined for REPL support #37

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

juxtin
Copy link

@juxtin juxtin commented Oct 13, 2017

The intent here is to fix Issue #7 without any further manual workarounds.

Previously, attempting to start elm-repl for a module that depends on Navigation would fail because document was undefined. This commit uses a dummy (empty object) value for the offending reference, preventing a crash.

@process-bot
Copy link

Thanks for the pull request! Make sure it satisfies this checklist. My human colleagues will appreciate it!

Here is what to expect next, and if anyone wants to comment, keep these things in mind.

Previously, attempting to start elm-repl for a module that depends on
Navigation would fail because `document` was undefined. This commit uses
a dummy (empty object) value for the offending reference, preventing a
crash.
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