Skip to content
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

Fix Website Build #2526

Merged
merged 8 commits into from
Sep 6, 2017
Merged

Conversation

spencercarli
Copy link
Member

Should fix website build so we can get doc updates live.

Test Plan

Follow instructions in website/README.md to build the site.

Fixes ReferenceError when not in the React Native environment.
@@ -24,7 +24,7 @@

var validateFormat = function(format) {};

if (__DEV__) {
if (global.__DEV__) {

This comment was marked as abuse.

This comment was marked as abuse.

This comment was marked as abuse.

@matthamil
Copy link
Member

matthamil commented Sep 5, 2017

Looks like it built correctly on my machine 🎉, awesome 😄 ! Not sure about the implications of changing __DEV__ to global.__DEV__ though.

@spencercarli spencercarli merged commit ba3fb1a into react-navigation:master Sep 6, 2017
@spencercarli spencercarli deleted the fix-docs branch October 18, 2017 00:46
sourcecode911 pushed a commit to sourcecode911/react-navigation that referenced this pull request Mar 9, 2020
* Update .eslintrc to allow compile.

* Resolve .web.js extension

* Look for __DEV__ on global

Fixes ReferenceError when not in the React Native environment.

* Replace __DEV__ at compile time via babel

* Don't try to lint a website build
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants