Skip to content

fix CONFIG_FILE ReferenceError#3

Merged
dbrans merged 1 commit into
dbrans:masterfrom
leetreveil:config_file_bug
Jan 10, 2013
Merged

fix CONFIG_FILE ReferenceError#3
dbrans merged 1 commit into
dbrans:masterfrom
leetreveil:config_file_bug

Conversation

@leetreveil

@leetreveil leetreveil commented Jan 10, 2013

Copy link
Copy Markdown
Contributor

CONFIG_FILE should be at the top level and not inside var config = (function...

Otherwise we get:

ReferenceError: CONFIG_FILE is not defined
    at tunnelAction (/Users/leetreveil/Dropbox/browserstack-cli/bin/cli.js:380:95)
    at Object.runAction [as _onTimeout] (/Users/leetreveil/Dropbox/browserstack-cli/bin/cli.js:455:10)
    at Timer.list.ontimeout (timers.js:101:19)

dbrans added a commit that referenced this pull request Jan 10, 2013
@dbrans
Merge pull request #3 from leetreveil/config_file_bug
3929724 
fix CONFIG_FILE ReferenceError
@dbrans dbrans merged commit 3929724 into dbrans:master Jan 10, 2013
@leetreveil leetreveil deleted the config_file_bug branch January 11, 2013 01:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@leetreveil @dbrans