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 the layer tests on node #660

Merged
merged 2 commits into from
May 24, 2017
Merged

Fix the layer tests on node #660

merged 2 commits into from
May 24, 2017

Conversation

howtimeflies0
Copy link

Fix the issue that headless GL context is not created properly on node

Copy link
Collaborator

@ibgreen ibgreen left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just curious, why was this needed?

@howtimeflies0
Copy link
Author

@ibgreen yeah... you are right. It's not needed. The real problem is npm run test-dist runs 'node-dist.js` and it doesn't include './setup-gl.js'

@howtimeflies0 howtimeflies0 force-pushed the fix-layer-test-node branch 3 times, most recently from c8f2ff9 to d69a239 Compare May 24, 2017 17:57
@ericsoco
Copy link
Contributor

Just tested locally after rebasing on master, looks good. Pushed the rebase up. Merge at will

@howtimeflies0 howtimeflies0 merged commit 964ddee into master May 24, 2017
Firenze11 pushed a commit to Firenze11/deck.gl that referenced this pull request May 31, 2017
* Require setup-gl.js in test-dist.js
@balthazar balthazar deleted the fix-layer-test-node branch June 15, 2017 19:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants